Skip to content

make the method setBackOffFunction work for the batch processing as well (backport to the branch 3.3.x)#4467

Open
LeosBitto wants to merge 2 commits into
spring-projects:3.3.xfrom
LeosBitto:3.3.x
Open

make the method setBackOffFunction work for the batch processing as well (backport to the branch 3.3.x)#4467
LeosBitto wants to merge 2 commits into
spring-projects:3.3.xfrom
LeosBitto:3.3.x

Conversation

@LeosBitto
Copy link
Copy Markdown
Contributor

This is a backport of the commit 7c9f5c0 to the branch 3.3.x which does not modify any existing public API. The only addition to the public API is a new overload of the method retryBatch in ErrorHandlingUtils which supports the implementation of a custom BackOffFunction and resetStateOnExceptionChange for the batch processing.

Signed-off-by: Leos Bitto <leos.bitto@gmail.com>
* @param resetStateOnExceptionChange true if a different exception thrown during retry should end this method.
* @return a new exception if resetStateOnExceptionChange was true and a different exception has occurred
* (retry with a possibly different BackOffExecution is expected if non-null is returned)
* @since 4.0.7
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The problem with this is that we have the same method on 4.1.x and there we use the since tag as 4.1.0, which is kind of moot since we are introducing this method all way back to 3.3.x. What do you think?

Copy link
Copy Markdown
Contributor Author

@LeosBitto LeosBitto Jun 5, 2026

Choose a reason for hiding this comment

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

I have changed it to include the versions with backport. If you agree, the same change should be made to main as well - there should be no need to explicitly mention 4.1.0, because all versions after 4.0.6 contain the change.

Signed-off-by: Leos Bitto <leos.bitto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants