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: set wait timeout and update default attempt timeout to 30 minutes #1779

Merged
merged 2 commits into from
Jun 6, 2023

Conversation

mutianf
Copy link
Contributor

@mutianf mutianf commented Jun 6, 2023

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.

@mutianf mutianf requested a review from a team as a code owner June 6, 2023 15:01
@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 Jun 6, 2023
@mutianf mutianf added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@mutianf mutianf added the automerge Merge the pull request once unit tests and other checks pass. label Jun 6, 2023
@gcf-owl-bot gcf-owl-bot bot requested a review from a team as a code owner June 6, 2023 17:31
@gcf-merge-on-green gcf-merge-on-green bot merged commit 11019b8 into googleapis:main Jun 6, 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 Jun 6, 2023
@mutianf mutianf deleted the update-timeout branch June 6, 2023 18:04
@mcookhome
Copy link

mcookhome commented Jun 28, 2023

hey @mutianf, sorry to come out of the woodwork, but when my team at work upgraded to use this version of java bigtable, all of our readRows retry settings (which were not respected before this version bump, surprisingly) started to be respected.

This caused an incident for us as we were unaware that our service was going to start respecting the timeouts. the only upgrade in the java cloud bom diff that seems relevant at all is the java bigtable diff, and this change seems somewhat related, but the retry settings we're seeing reflected don't really have much to do with this change.

Do you know any other changes that may have caused our bigtable readRows retry settings to begin to be respected? Thanks for any details you may provide!

@igorbernstein2
Copy link
Contributor

Hi @mcookhome,

The relevant change was here: googleapis/sdk-platform-java#1473. I apologize for the disruption

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.

3 participants