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

Remove composite retryPolicy when both limit and policy set #4008

Conversation

acktsap
Copy link
Contributor

@acktsap acktsap commented Sep 29, 2021

According to comment in retryLimit, it should be ignored when explicit retryPolicy is set.

/**
* The maximum number of times to try a failed item. Zero and one both translate to try only once and do not retry.
* Ignored if an explicit {@link #retryPolicy} is set.
*
* @param retryLimit the retry limit (default 0)
* @return this for fluent chaining
*/
public FaultTolerantStepBuilder<I, O> retryLimit(int retryLimit) {
this.retryLimit = retryLimit;
return this;
}

@acktsap acktsap force-pushed the topic/use-retrylimit-and-exception-both branch from 204e00d to cf2e975 Compare September 29, 2021 21:44
@acktsap acktsap closed this Nov 24, 2022
@acktsap acktsap deleted the topic/use-retrylimit-and-exception-both branch February 6, 2024 03:15
@acktsap acktsap restored the topic/use-retrylimit-and-exception-both branch February 6, 2024 03:15
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.

None yet

1 participant