funky-eyes opened a new issue, #7568: URL: https://github.com/apache/incubator-seata/issues/7568
### Check Ahead - [x] I have searched the [issues](https://github.com/seata/seata/issues) of this repository and believe that this is not a duplicate. - [x] I am willing to try to fix this bug myself. ### Ⅰ. Issue Description https://github.com/apache/incubator-seata/blob/2.x/integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/handler/AbstractProxyInvocationHandler.java https://github.com/apache/incubator-seata/blob/2.x/integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/parser/DefaultInterfaceParser.java#L84 `Collections.sort(invocationHandlerList, Comparator.comparingInt(ProxyInvocationHandler::order));` The method order is the one actually used for sorting, not getOrder. Therefore, the order in AbstractProxyInvocationHandler is entirely unnecessary. I suggest setting the value of order in AbstractProxyInvocationHandler to 0 and implementing the order method in the parent class ProxyInvocationHandler. This approach ensures backward compatibility and prevents issues with aspect order inconsistencies when directly implementing the order method in the parent class. ### Ⅱ. Describe what happened _No response_ ### Ⅲ. Describe what you expected to happen _No response_ ### Ⅳ. How to reproduce it (as minimally and precisely as possible) _No response_ ### Ⅴ. Anything else we need to know? _No response_ ### Ⅵ. Environment _No response_ -- 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.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
