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

optimize: report the tcc fence transaction isolation level #6768

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 64.28571% with 5 lines in your changes missing coverage. Please review.

Project coverage is 52.34%. Comparing base (79a6744) to head (b5c1801). Report is 1 commits behind head on 2.x.

Files Patch % Lines
...ava/org/apache/seata/rm/tcc/utils/MethodUtils.java 66.66% 3 Missing :warning:
...m/tcc/interceptor/TccActionInterceptorHandler.java 60.00% 1 Missing and 1 partial :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/apache/incubator-seata/pull/6768/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/6768?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 #6768 +/- ## ============================================ + Coverage 52.17% 52.34% +0.16% - Complexity 6288 6304 +16 ============================================ Files 1063 1064 +1 Lines 37153 37167 +14 Branches 4408 4407 -1 ============================================ + Hits 19385 19454 +69 + Misses 15856 15795 -61 - Partials 1912 1918 +6 ``` | [Files](https://app.codecov.io/gh/apache/incubator-seata/pull/6768?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/6768?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% <ø> (ø)` | | | [...m/tcc/interceptor/TccActionInterceptorHandler.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6768?src=pr&el=tree&filepath=tcc%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Frm%2Ftcc%2Finterceptor%2FTccActionInterceptorHandler.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dGNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zZWF0YS9ybS90Y2MvaW50ZXJjZXB0b3IvVGNjQWN0aW9uSW50ZXJjZXB0b3JIYW5kbGVyLmphdmE=) | `66.66% <60.00%> (-0.48%)` | :arrow_down: | | [...ava/org/apache/seata/rm/tcc/utils/MethodUtils.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6768?src=pr&el=tree&filepath=tcc%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Frm%2Ftcc%2Futils%2FMethodUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dGNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zZWF0YS9ybS90Y2MvdXRpbHMvTWV0aG9kVXRpbHMuamF2YQ==) | `66.66% <66.66%> (ø)` | | ... and [8 files with indirect coverage changes](https://app.codecov.io/gh/apache/incubator-seata/pull/6768/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)