Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

feat: delete deprecated Dataproc v1beta2 API client #253

Merged

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Aug 31, 2021

See #249. Googlers see internal bug b/197576296.

Committer: @medb
PiperOrigin-RevId: 392723737

Source-Link: googleapis/googleapis@8bf048d

Source-Link: https://github.com/googleapis/googleapis-gen/commit/f45e49bb22b39098ca73b18228b2014ab931d38a

@parthea parthea added kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 31, 2021
@parthea parthea requested a review from a team as a code owner August 31, 2021 10:50
@product-auto-label product-auto-label bot added the api: dataproc Issues related to the googleapis/python-dataproc API. label Aug 31, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 31, 2021
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 31, 2021
@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 31, 2021
@parthea
Copy link
Contributor Author

parthea commented Aug 31, 2021

@leahecole Please could you review/approve?

@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 31, 2021
@busunkim96
Copy link
Contributor

The changes LGTM. Is the intent to take a major bump in this library?

@parthea
Copy link
Contributor Author

parthea commented Aug 31, 2021

Yes, I'd like to take a major bump. I'll mark this PR as Do Not Merge while I confirm with the broader team.

@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 31, 2021
@leahecole
Copy link
Contributor

I think it's fine by me but also want to cc @loferris - they're working on Dataproc stuff here and there and I want to make sure they see this too

@parthea parthea assigned loferris and unassigned leahecole Sep 1, 2021
@parthea parthea added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 1, 2021
@parthea parthea added the kokoro:run Add this label to force Kokoro to re-run the tests. label Sep 1, 2021
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Sep 1, 2021
@parthea parthea changed the title feat!: delete deprecated Dataproc v1beta2 API client feat: delete deprecated Dataproc v1beta2 API client Sep 1, 2021
@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 1, 2021
@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Sep 1, 2021
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Sep 1, 2021
@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Sep 2, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit b0db6da into main Sep 2, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the owl-bot-f74e8a85-e49b-419e-b9b3-ab196ba40aff branch September 2, 2021 00:14
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 2, 2021
@parthea parthea mentioned this pull request Sep 25, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 29, 2021
Closes #271 

We need to bump the major version as the dataproc v1beta2 client was deleted in #253. The last version that contains the v1beta2 client is version 2.5.0. See #271 for more information.

feat: delete deprecated Dataproc v1beta2 API client
docs: update cluster sample

Release-As: 3.0.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: dataproc Issues related to the googleapis/python-dataproc API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants