Bughue opened a new pull request, #4555: URL: https://github.com/apache/incubator-seata/pull/4555
<!-- Please make sure you have read and understood the contributing guidelines --> - [ ] I have registered the PR [changes](https://github.com/seata/seata/tree/develop/changes). ### Ⅰ. Describe what this PR did 为TransactionHook接口新增参数，目前只有xid，本来想把GlobalTransaction和TransactionInfo都传进去的，但TransactionInfo里面信息现在利用率比较低（方法拿到这个参数后里面也没有很多关键信息），而GlobalTransaction传给hook的话权限过大，用户有可能会对transaction进行更改，感觉还是谨慎开放的好 ### Ⅱ. Does this pull request fix one issue? <!-- If that, add "fixes #xxx" below in the next line, for example, fixes #97. --> fix https://github.com/seata/seata/issues/4554 ### Ⅲ. Why don't you add test cases (unit test/integration test)? ### Ⅳ. Describe how to verify it ### Ⅴ. Special notes for reviews -- 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
