Copilot commented on code in PR #7570:
URL: https://github.com/apache/incubator-seata/pull/7570#discussion_r2262026862
##########
integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/handler/AbstractProxyInvocationHandler.java:
##########
@@ -28,7 +28,7 @@ public abstract class AbstractProxyInvocationHandler
implements ProxyInvocationH
protected abstract Object doInvoke(InvocationWrapper invocation) throws
Throwable;
- protected int order = Integer.MAX_VALUE;
+ protected int order = 0;
Review Comment:
Changing the default order value from Integer.MAX_VALUE to 0 is a breaking
change that could affect existing handler ordering. Consider the impact on
handlers that relied on the previous default behavior where unspecified
handlers would be executed last.
```suggestion
protected int order = Integer.MAX_VALUE;
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org
For additional commands, e-mail: notifications-h...@seata.apache.org
