Skip to content

Commit

Permalink
Change default order for @EnableRetry
Browse files Browse the repository at this point in the history
The new default order is `Ordered.LOWEST_PRECEDENCE - 1` to make sure
the advice is applied before other advices with
`Ordered.LOWEST_PRECEDENCE` order (e.g. an advice responsible
for `@Transactional` behavior).
  • Loading branch information
xak2000 committed Mar 8, 2023
1 parent debc4ad commit 9c4eada
Showing 1 changed file with 7 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
*
* @author Dave Syer
* @author Yanming Zhou
* @author Ruslan Stelmachenko
* @since 1.1
*
*/
Expand All @@ -54,12 +55,13 @@
boolean proxyTargetClass() default false;

/**
* Indicate the order in which the {@link RetryConfiguration} should be applied.
* Indicate the order in which the {@link RetryConfiguration} AOP <b>advice</b> should
* be applied.
* <p>
* The default is {@link Ordered#LOWEST_PRECEDENCE} in order to run after all other
* post-processors, so that it can add an advisor to existing proxies rather than
* double-proxy.
* The default is {@code Ordered.LOWEST_PRECEDENCE - 1} in order to make sure the
* advice is applied before other advices with {@link Ordered#LOWEST_PRECEDENCE} order
* (e.g. an advice responsible for {@code @Transactional} behavior).
*/
int order() default Ordered.LOWEST_PRECEDENCE;
int order() default Ordered.LOWEST_PRECEDENCE - 1;

}

0 comments on commit 9c4eada

Please sign in to comment.