这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@BenWhitehead
Copy link
Collaborator

If the executor service provided to the DefaultRetryContext is shutdown and an onSuccess attempts to be scheduled a RejectedExceptionException will be thrown.

Now, we catch and add it as a suppressed exception provided to recordError and immediately call onFailure, as we are no longer able to proceed with scheduling the retry.

…Exception to onFailure

If the executor service provided to the DefaultRetryContext is shutdown and an onSuccess attempts to be scheduled a RejectedExceptionException will be thrown.

Now, we catch and add it as a suppressed exception provided to recordError and immediately call onFailure, as we are no longer able to proceed with scheduling the retry.
@BenWhitehead BenWhitehead requested a review from a team as a code owner October 3, 2025 18:03
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/java-storage API. labels Oct 3, 2025
@BenWhitehead BenWhitehead merged commit 1be31bd into main Oct 6, 2025
25 checks passed
@BenWhitehead BenWhitehead deleted the fix/retry-context/catch-ree branch October 6, 2025 15:48
Copy link

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants