Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add sample for updating a cluster in Dataproc. #362

Closed
wants to merge 1 commit into from
Closed

Add sample for updating a cluster in Dataproc. #362

wants to merge 1 commit into from

Conversation

sudeepag
Copy link

No description provided.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label May 27, 2016
@sudeepag
Copy link
Author

Signed the CLA.

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels May 27, 2016
waprin
waprin reviewed May 28, 2016
View reviewed changes
dataproc/create_cluster_and_submit_job.py
Use 'config.worker_config.num_instances' and 'workerConfig' for workers.
Use 'config.secondary_worker_config.num_instances' and
'secondaryWorkerConfig' for preemptible workers."""
print('Updating cluster to %d workers.' % num_workers)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use {} and format() rather then % for string interpolation in this repo

@waprin
Copy link
Contributor

waprin commented May 28, 2016

Thanks for the PR. I think it'd be better to a) move this into a totally separate file with a main function b) write an integration test that verifies the changes to the resource was made.

@waprin
Copy link
Contributor

waprin commented Jun 23, 2016

closing based on lack of author response, feel free to re-open.

@waprin waprin closed this Jun 23, 2016
nicain pushed a commit that referenced this pull request Nov 2, 2022
@renovate-bot
chore(deps): update dependency google-cloud-documentai to v1.5.1 (#362)
8695324
@nicain nicain mentioned this pull request Nov 2, 2022
Closed
8 tasks
msampathkumar pushed a commit that referenced this pull request Nov 14, 2022
@renovate-bot
chore(deps): update dependency google-cloud-dataproc to v4.0.1 (#362)
c3ac64a
@msampathkumar msampathkumar mentioned this pull request Nov 14, 2022
Merged
8 tasks
@telpirion telpirion mentioned this pull request Nov 16, 2022
Merged
8 tasks
telpirion pushed a commit that referenced this pull request Nov 16, 2022
@renovate-bot @parthea
chore(deps): update dependency google-cloud-vision to v2.7.2 (#362)
368c217 
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
arbrown pushed a commit that referenced this pull request Nov 17, 2022
@gcf-owl-bot
chore(python): use black==22.3.0 (#362)
a0e7f43 
Source-Link: googleapis/synthtool@6fab84a
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:7cffbc10910c3ab1b852c05114a08d374c195a81cdec1d4a67a1d129331d0bfe

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@arbrown arbrown mentioned this pull request Nov 17, 2022
Merged
8 tasks
dandhlee pushed a commit that referenced this pull request Nov 17, 2022
@gcf-owl-bot @dandhlee
chore(python): use black==22.3.0 (#362)
ebff875 
Source-Link: googleapis/synthtool@6fab84a
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:7cffbc10910c3ab1b852c05114a08d374c195a81cdec1d4a67a1d129331d0bfe

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
dandhlee pushed a commit that referenced this pull request Nov 18, 2022
@renovate-bot @parthea @dandhlee
chore(deps): update dependency google-cloud-vision to v2.7.2 (#362)
e46ed71 
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
holtskinner pushed a commit that referenced this pull request Jan 3, 2023
@renovate-bot @holtskinner
chore(deps): update dependency google-cloud-documentai to v1.5.1 (#362)
c00fd29
@holtskinner holtskinner mentioned this pull request Jan 4, 2023
Merged
8 tasks
holtskinner pushed a commit that referenced this pull request Jan 4, 2023
@renovate-bot
chore(deps): update dependency google-cloud-documentai to v1.5.1 (#362)
410a444
dandhlee pushed a commit that referenced this pull request Jan 5, 2023
@renovate-bot @dandhlee
chore(deps): update dependency google-cloud-documentai to v1.5.1 (#362)
a985bea
@telpirion telpirion mentioned this pull request Jan 13, 2023
Merged
8 tasks
telpirion pushed a commit that referenced this pull request Jan 13, 2023
@gcf-owl-bot
chore(python): add nox session to sort python imports (#362)
9525dea 
Source-Link: googleapis/synthtool@1b71c10
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:00c9d764fd1cd56265f12a5ef4b99a0c9e87cf261018099141e2ca5158890416

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
dandhlee pushed a commit that referenced this pull request Feb 9, 2023
@gcf-owl-bot @dandhlee
chore(python): add nox session to sort python imports (#362)
61965ea 
Source-Link: googleapis/synthtool@1b71c10
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:00c9d764fd1cd56265f12a5ef4b99a0c9e87cf261018099141e2ca5158890416

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
telpirion pushed a commit that referenced this pull request Mar 13, 2023
@gcf-owl-bot @telpirion
chore(python): add nox session to sort python imports (#362)
154a375 
Source-Link: googleapis/synthtool@1b71c10
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:00c9d764fd1cd56265f12a5ef4b99a0c9e87cf261018099141e2ca5158890416

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sudeepag @googlebot @waprin