YongGoose commented on issue #7568: URL: https://github.com/apache/incubator-seata/issues/7568#issuecomment-3149449985
> I also hope to make the order variable fully functional. However, the current default value of order is Integer.MAX_VALUE. After reading the code, I noticed that the three current implementations—SagaAnnotationActionInterceptorHandler and TccActionInterceptorHandler—have already overridden the order method. If the third implementation, GlobalTransactionalInterceptorHandler, also overrides the order method and sets the order value to 0, we can then fully enable the order in AbstractProxyInvocationHandler. This approach will not disrupt the existing order and will allow future implementations to take advantage of the order method in AbstractProxyInvocationHandler. > > Additionally, I’m thrilled to hear this news, as it demonstrates that the openness and inclusiveness of the Seata community are being recognized by international contributors. Perhaps in the future, the community might even have the chance to host an international meetup, either online or offline—who knows! That sounds like a good approach. In that case, how about overriding the `order` method in `GlobalTransactionalInterceptorHandler` as well, and having the `setOrder` and `getOrder` methods in `SeataInterceptor` set and return the `order` field from `AbstractProxyInvocationHandler`? Instead of defining `order` as a default method in `ProxyInvocationHandler`, we could implement it in `AbstractProxyInvocationHandler` and set the default value of the `order` field to 0. What do you think? > Additionally, I’m thrilled to hear this news, as it demonstrates that the openness and inclusiveness of the Seata community are being recognized by international contributors. Perhaps in the future, the community might even have the chance to host an international meetup, either online or offline—who knows! I’m really excited too! It would be great if we could give a talk together at an international meetup someday. 🚀 -- 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
