search

apache / incubator-seata

:fire: Seata is an easy-to-use, high-performance, open source distributed transaction solution.
https://seata.apache.org/
Apache License 2.0
25.25k stars 8.77k forks source link

bugfix: fix tcc dead lock #6769

Closed iAmClever closed 1 month ago

iAmClever commented 1 month ago

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #6679

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 13.33333% with 26 lines in your changes missing coverage. Please review.

Project coverage is 52.14%. Comparing base (5617405) to head (b52c30e). Report is 1 commits behind head on 2.x.

Files Patch % Lines
.../org/apache/seata/rm/fence/SpringFenceHandler.java 0.00% 26 Missing :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/apache/incubator-seata/pull/6769/graphs/tree.svg?width=650&height=150&src=pr&token=tbmHt2ZfxO&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)](https://app.codecov.io/gh/apache/incubator-seata/pull/6769?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) ```diff @@ Coverage Diff @@ ## 2.x #6769 +/- ## ============================================ - Coverage 52.34% 52.14% -0.21% + Complexity 6304 6287 -17 ============================================ Files 1064 1064 Lines 37167 37194 +27 Branches 4409 4412 +3 ============================================ - Hits 19456 19395 -61 - Misses 15793 15884 +91 + Partials 1918 1915 -3 ``` | [Files](https://app.codecov.io/gh/apache/incubator-seata/pull/6769?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [...c/main/java/org/apache/seata/common/Constants.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6769?src=pr&el=tree&filepath=common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Fcommon%2FConstants.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Y29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zZWF0YS9jb21tb24vQ29uc3RhbnRzLmphdmE=) | `100.00% <ø> (ø)` | | | [...va/org/apache/seata/rm/tcc/TCCResourceManager.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6769?src=pr&el=tree&filepath=tcc%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Frm%2Ftcc%2FTCCResourceManager.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dGNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zZWF0YS9ybS90Y2MvVENDUmVzb3VyY2VNYW5hZ2VyLmphdmE=) | `50.00% <100.00%> (+2.32%)` | :arrow_up: | | [.../org/apache/seata/rm/fence/SpringFenceHandler.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6769?src=pr&el=tree&filepath=spring%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Frm%2Ffence%2FSpringFenceHandler.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3ByaW5nL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zZWF0YS9ybS9mZW5jZS9TcHJpbmdGZW5jZUhhbmRsZXIuamF2YQ==) | `0.00% <0.00%> (ø)` | | ... and [9 files with indirect coverage changes](https://app.codecov.io/gh/apache/incubator-seata/pull/6769/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)