Skip to content

fix: Modify ConvertExceptionCallable to retry on Goaway#1588

Merged
gcf-merge-on-green[bot] merged 4 commits intogoogleapis:mainfrom
jackdingilian:retry-on-goaway
Feb 7, 2023
Merged

fix: Modify ConvertExceptionCallable to retry on Goaway#1588
gcf-merge-on-green[bot] merged 4 commits intogoogleapis:mainfrom
jackdingilian:retry-on-goaway

Conversation

@jackdingilian
Copy link
Copy Markdown
Contributor

@jackdingilian jackdingilian commented Jan 19, 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.

@jackdingilian jackdingilian requested a review from a team January 19, 2023 17:24
@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 Jan 19, 2023
@jackdingilian jackdingilian changed the title Modify ConvertExceptionCallable to retry on Goaway fix: Modify ConvertExceptionCallable to retry on Goaway Jan 19, 2023
@igorbernstein2 igorbernstein2 added the owlbot:run Add this label to trigger the Owlbot post processor. label Feb 2, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Feb 2, 2023
Copy link
Copy Markdown
Contributor

@igorbernstein2 igorbernstein2 left a comment

Choose a reason for hiding this comment

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

lgtm

@igorbernstein2 igorbernstein2 added the automerge Merge the pull request once unit tests and other checks pass. label Feb 2, 2023
@gcf-owl-bot gcf-owl-bot bot requested a review from a team February 2, 2023 18:47
@gcf-merge-on-green
Copy link
Copy Markdown

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 Feb 3, 2023
@igorbernstein2 igorbernstein2 added automerge Merge the pull request once unit tests and other checks pass. owlbot:run Add this label to trigger the Owlbot post processor. labels Feb 7, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Feb 7, 2023
@gcf-merge-on-green gcf-merge-on-green bot merged commit cf752ea into googleapis:main Feb 7, 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 Feb 7, 2023
gcf-merge-on-green bot pushed a commit that referenced this pull request Feb 10, 2023
🤖 I have created a release *beep* *boop*
---


## [2.19.0](https://togithub.com/googleapis/java-bigtable/compare/v2.18.4...v2.19.0) (2023-02-10)


### Features

* Support Cloud Bigtable Changestream ([#1569](https://togithub.com/googleapis/java-bigtable/issues/1569)) ([c7b4fdf](https://togithub.com/googleapis/java-bigtable/commit/c7b4fdf70f02b89f178a130d7fdcfcf8b2a6b790))


### Bug Fixes

* Modify ConvertExceptionCallable to retry on Goaway ([#1588](https://togithub.com/googleapis/java-bigtable/issues/1588)) ([cf752ea](https://togithub.com/googleapis/java-bigtable/commit/cf752eaf3b5fb2cddb11a2fbb5acb5a72c9862c2))


### Documentation

* Fix javadoc code example for awaitOptimizeRestoredTableAsync ([#1617](https://togithub.com/googleapis/java-bigtable/issues/1617)) ([8b23bb9](https://togithub.com/googleapis/java-bigtable/commit/8b23bb93d40dbfd9591c7d7546216e2543f9f418))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
tonytanger pushed a commit to tonytanger/java-bigtable that referenced this pull request Mar 20, 2023
…googleapis#1588) (googleapis#1415)

* chore: pin versions of dependencies for compatibility with Python 3.6

* fix path of requirements file in synthtool
Source-Link: googleapis/synthtool@69cdb47
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:6566dc8226f20993af18e5a4e7a2b1ba85a292b02dedb6a1634cf10e1b418fa5
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.

2 participants