YongGoose commented on PR #7587: URL: https://github.com/apache/incubator-seata/pull/7587#issuecomment-3194906461
In the code below, the execution order of the `Seata interceptor` has already been adjusted to be one lower than Spring’s `TransactionInterceptor`. - https://github.com/apache/incubator-seata/blob/b0566047717a485ba0b62e16d8724d50ffe77e6c/spring/src/main/java/org/apache/seata/spring/annotation/GlobalTransactionScanner.java#L449 That’s why I don’t think this change is necessary. Personally, I also prefer keeping the code clear. If we end up having two different `order` values, it might make future maintenance more complicated. WDYT? -- 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
