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.

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

fix: if new_partitions is size 0, do not enforce size check #1673

Merged

Conversation

tonytanger
Copy link
Contributor

Do not enforce new_partitions and change_stream_continuation_tokens to be the same size if new_partitions has size of 0 because Cloud Bigtable backend may not be updated to serve new_partitions field yet.

new_partitions is a new field and the backend may not be serving this field.

Change-Id: Id21c293b92c304f05b905ca8e8b3988b9241866e

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

@tonytanger tonytanger requested a review from a team as a code owner March 20, 2023 20:14
@conventional-commit-lint-gcf
Copy link

conventional-commit-lint-gcf bot commented Mar 20, 2023

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@product-auto-label product-auto-label bot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Mar 20, 2023
@tonytanger tonytanger changed the title If new_partitions is size 0, do not enforce size check. fix: If new_partitions is size 0, do not enforce size check. Mar 20, 2023
@tonytanger
Copy link
Contributor Author

@tengzhonger

Change-Id: I4e9b6507ff63df9f9ec9d57a0fc34e68e16f6e30
@tonytanger tonytanger force-pushed the remove_new_partitions_check branch from d905e4f to 549e495 Compare March 20, 2023 20:22
@tonytanger tonytanger changed the title fix: If new_partitions is size 0, do not enforce size check. fix: if new_partitions is size 0, do not enforce size check Mar 20, 2023
@mutianf mutianf added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
@mutianf mutianf added the automerge Merge the pull request once unit tests and other checks pass. label Mar 20, 2023
@gcf-merge-on-green
Copy link

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Mar 21, 2023
@kolea2 kolea2 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 21, 2023
@yoshi-kokoro yoshi-kokoro removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Mar 21, 2023
@kolea2 kolea2 added the automerge Merge the pull request once unit tests and other checks pass. label Mar 21, 2023
@gcf-merge-on-green gcf-merge-on-green bot merged commit 07bcfd9 into googleapis:main Mar 21, 2023
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Mar 21, 2023
gcf-merge-on-green bot pushed a commit that referenced this pull request Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the googleapis/java-bigtable API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants