optimize: build UndoItem logic

[xiaochangbai]

• I have registered the PR changes.

Ⅰ. Describe what this PR did

optimize build UndoItem logic

Ⅱ. Does this pull request fix one issue?

fixes #4280

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[xiaochangbai]

不知道这样改对不对，大佬看了之后麻烦指点一下哈

[codecov-commenter]

Codecov Report

Merging #4282 (b791299) into develop (250bc70) will decrease coverage by 0.03%.
The diff coverage is 12.50%.

@@              Coverage Diff              @@
##             develop    #4282      +/-   ##
=============================================
- Coverage      49.02%   48.99%   -0.04%     
+ Complexity      3955     3950       -5     
=============================================
  Files            733      733              
  Lines          24974    24978       +4     
  Branches        3089     3092       +3     
=============================================
- Hits           12243    12237       -6     
- Misses         11434    11442       +8     
- Partials        1297     1299       +2     

Impacted Files	Coverage Δ
...source/exec/mysql/MySQLInsertOrUpdateExecutor.java	31.07% <0.00%> (-0.36%)	⬇️
...ta/rm/datasource/exec/AbstractDMLBaseExecutor.java	54.71% <25.00%> (-4.11%)	⬇️
...n/src/main/java/io/seata/common/util/IdWorker.java	77.08% <0.00%> (-6.25%)	⬇️
.../rm/datasource/exec/BaseTransactionalExecutor.java	55.26% <0.00%> (-0.66%)	⬇️
...o/seata/server/coordinator/DefaultCoordinator.java	52.18% <0.00%> (-0.37%)	⬇️

[caohdgege]

LGTM

[funky-eyes]

LGTM

[codelearner1024]

acquire before image can also be moved into the if judgement. @xiaochangbai

[a1104321118]

#4280

hello， 这个这样判断有点问题，我在用 mybatis-plus 的时候，使用 saveBatch 方法，它底层用的是 Statement.executeBatch 方法，这个方法无法返回影响的行数，导致你这里的判断会失效，所以没有生成undo log；除非开启这个参数 ： [rewriteBatchedStatements] ，但是性能会下降很多

相关资料：
com.mysql.cj.jdbc.ClientPreparedStatement#executeBatchedInserts
https://forums.mysql.com/read.php?39,699815,699821#msg-699821

如果使用mybatis 原生的批量插入，底层调用的 是 Statement.execute 方法，这个方法可以获取到影响的行数，没有问题；

seata 客户端版本 1.5.1
mysql 8.0

[a1104321118]

您好  我已经收到你的邮件，谢谢！HR

[xiaochangbai]

#4280

hello， 这个这样判断有点问题，我在用 mybatis-plus 的时候，使用 saveBatch 方法，它底层用的是 Statement.executeBatch 方法，这个方法无法返回影响的行数，导致你这里的判断会失效，所以没有生成undo log；除非开启这个参数 ： [rewriteBatchedStatements] ，但是性能会下降很多

相关资料： com.mysql.cj.jdbc.ClientPreparedStatement#executeBatchedInserts https://forums.mysql.com/read.php?39,699815,699821#msg-699821

如果使用mybatis 原生的批量插入，底层调用的 是 Statement.execute 方法，这个方法可以获取到影响的行数，没有问题；

seata 客户端版本 1.5.1 mysql 8.0

好，这个问题我看看。感谢您的反馈