Thanks to visit codestin.com
Credit goes to github.com

Skip to content

optimize: add ExceptionUtil class for unwarp error msg #6666

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.

Sign up for GitHub

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

Jump to bottom
Merged
funky-eyes merged 6 commits into from
Jul 12, 2024
Merged

optimize: add ExceptionUtil class for unwarp error msg #6666

funky-eyes merged 6 commits into from
Jul 12, 2024

Conversation

@iAmClever
Copy link
Contributor

@iAmClever iAmClever commented Jul 12, 2024 •
edited by funky-eyes
Loading

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #6665

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@funky-eyes funky-eyes changed the title optimize: add ExceptionUtil class for unwarp error msg (#6665) optimize: add ExceptionUtil class for unwarp error msg Jul 12, 2024
funky-eyes
funky-eyes reviewed Jul 12, 2024
View reviewed changes
@funky-eyes funky-eyes added this to the 2.2.0 milestone Jul 12, 2024
@codecov
Copy link

codecov bot commented Jul 12, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.

Project coverage is 50.73%. Comparing base (cbbfa22) to head (f656b1e).
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6666      +/-   ##
============================================
- Coverage     50.75%   50.73%   -0.03%     
+ Complexity     5709     5705       -4     
============================================
  Files          1034     1035       +1     
  Lines         35854    35863       +9     
  Branches       4255     4255              
============================================
- Hits          18197    18194       -3     
- Misses        15838    15850      +12     
  Partials       1819     1819
Files Coverage Δ
...va/org/apache/seata/rm/tcc/TCCResourceManager.java 47.67% <0.00%> (ø)
...g/apache/seata/common/exception/ExceptionUtil.java 0.00% <0.00%> (ø)
.../org/apache/seata/rm/fence/SpringFenceHandler.java 0.00% <0.00%> (ø)

... and 10 files with indirect coverage changes

funky-eyes
funky-eyes approved these changes Jul 12, 2024
View reviewed changes
Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes added the first-time contributor first-time contributor label Jul 12, 2024
@funky-eyes
Copy link
Contributor

funky-eyes commented Jul 12, 2024

请把你的钉钉联系方式发送至：[email protected] 我将邀请你加入seata开发者群中
Please send me your DingTalk contact information to [email protected]. I will invite you to join the Seata developers group.

@funky-eyes funky-eyes merged commit 7739e50 into apache:2.x Jul 12, 2024
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
@iAmClever
optimize: add ExceptionUtil class for unwarp error msg (apache#6666)
1af8819
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@funky-eyes funky-eyes funky-eyes approved these changes

Assignees

No one assigned

Labels

first-time contributor first-time contributor module/tcc tcc module type: optimize

Projects

None yet

Milestone

2.2.0

Development

Successfully merging this pull request may close these issues.

TCC模式下，开启它的useTCCFence功能，在rollback方法中抛出一个异常，业务系统服务捕获到的异常信息为null

2 participants

@iAmClever @funky-eyes