funky-eyes commented on code in PR #7135: URL: https://github.com/apache/incubator-seata/pull/7135#discussion_r1943945921
########## changes/en-us/2.x.md: ########## @@ -17,6 +17,7 @@ Add changes here for all PR submitted to the 2.x branch. - [[#7112](https://github.com/apache/incubator-seata/pull/7112)] bugfix: remove the condition that IPv6 must start with fe80 - [[#7107](https://github.com/apache/incubator-seata/pull/7107)] fix the issue of failing to parse annotations in TCC mode when the business object is a proxy object. - [[#7124](https://github.com/apache/incubator-seata/pull/7124)] bugfix: GlobalTransactionScanner.afterPropertiesSet need do scanner check +- [[#7135](https://github.com/apache/incubator-seata/pull/7135)] catch SQLIntegrityConstraintViolationException and handle rollback Review Comment: ```suggestion - [[#7135](https://github.com/apache/incubator-seata/pull/7135)] treating a unique index conflict during rollback as a dirty write ``` ########## changes/zh-cn/2.x.md: ########## @@ -17,6 +17,7 @@ - [[#7112](https://github.com/apache/incubator-seata/pull/7112)] 校验是否IPv6网络ip取消必须以fe80开始的条件 - [[#7107](https://github.com/apache/incubator-seata/pull/7107)] 修复tcc模式下，当业务对象为代理对象时，解析注解失败问题。 - [[#7124](https://github.com/apache/incubator-seata/pull/7124)] GlobalTransactionScanner.afterPropertiesSet方法需要做扫描检查 +- [[#7135](https://github.com/apache/incubator-seata/pull/7135)] 修复捕获 SQLIntegrityConstraintViolationException 并处理回滚的问题 Review Comment: ```suggestion - [[#7135](https://github.com/apache/incubator-seata/pull/7135)] 回滚时遇到唯一索引冲突视为脏写 ``` -- 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
