-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix #819 add roll back on for GlobalTransaction #820
Conversation
…ure_tx_rollback # Conflicts: # spring/src/main/java/io/seata/spring/annotation/GlobalTransactionalInterceptor.java # tm/src/main/java/io/seata/tm/api/TransactionalTemplate.java # tm/src/test/java/io/seata/tm/api/TransactionTemplateTest.java
…ure_tx_rollback # Conflicts: # spring/src/main/java/io/seata/spring/annotation/GlobalTransactionalInterceptor.java # tm/src/main/java/io/seata/tm/api/TransactionalTemplate.java # tm/src/test/java/io/seata/tm/api/TransactionTemplateTest.java
Codecov Report
@@ Coverage Diff @@
## develop #820 +/- ##
============================================
+ Coverage 37.74% 38.2% +0.46%
- Complexity 974 1008 +34
============================================
Files 217 220 +3
Lines 8364 8446 +82
Branches 1002 1016 +14
============================================
+ Hits 3157 3227 +70
- Misses 4826 4835 +9
- Partials 381 384 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some comment
spring/src/main/java/io/seata/spring/annotation/GlobalTransactionalInterceptor.java
Outdated
Show resolved
Hide resolved
spring/src/main/java/io/seata/spring/annotation/GlobalTransactionalInterceptor.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's ok to me,except for a minor grammar problem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
* fix apache#819 add roll back on for GlobalTransaction * Merge branch 'develop' of https://github.com/alibaba/fescar into feature_tx_rollback # Conflicts: # spring/src/main/java/io/seata/spring/annotation/GlobalTransactionalInterceptor.java # tm/src/main/java/io/seata/tm/api/TransactionalTemplate.java # tm/src/test/java/io/seata/tm/api/TransactionTemplateTest.java * format code and add test * format code and add test * for review and thanks reviewers * update pkg
Ⅰ. Describe what this PR did
add roll back on for GlobalTransaction
roll back when we throw the exception what we want .
Ⅱ. Does this pull request fix one issue?
fix #819
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews