optimize: branch transaction repeated reporting false

[zjinlei]

Ⅰ. Describe what this PR did

Downstream processed branch status reporting, and only needs to be rollbacked local transaction when LOCK_RETRY_POLICY_BRANCH_ROLLBACK_ON_CONFLICT is false

Ⅱ. Does this pull request fix one issue?

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[slievrly]

https://github.com/ggndnn/seata/blob/14c9f3a33474e246bb6662d06441c87f613978e7/rm-datasource/src/main/java/io/seata/rm/datasource/ConnectionProxy.java#L224 if this line throw SQLException will duplicate report?

[codecov-io]

Codecov Report

Merging #1633 into develop will decrease coverage by 0.01%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop    #1633      +/-   ##
=============================================
- Coverage      47.39%   47.38%   -0.02%     
- Complexity      1777     1778       +1     
=============================================
  Files            362      362              
  Lines          13151    13153       +2     
  Branches        1626     1627       +1     
=============================================
- Hits            6233     6232       -1     
- Misses          6245     6247       +2     
- Partials         673      674       +1

Impacted Files	Coverage Δ	Complexity Δ
...ta/rm/datasource/exec/AbstractDMLBaseExecutor.java	73.52% <100%> (+1.65%)	7 <0> (+1)	⬆️
...server/store/file/FileTransactionStoreManager.java	45.99% <0%> (-1.05%)	19% <0%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 520c5ca...bf353cf. Read the comment docs.

[slievrly]

It’s my fault, LGTM.

[xingfudeshi]

LGTM.
UT is passed.