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

Skip to content

Conversation

@renliangyu857
Copy link
Contributor

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #5206

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-commenter
Copy link

codecov-commenter commented Dec 28, 2022

Codecov Report

Merging #5212 (8b640b7) into develop (1bad3b8) will decrease coverage by 0.05%.
The diff coverage is 25.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #5212      +/-   ##
=============================================
- Coverage      48.49%   48.44%   -0.06%     
+ Complexity      4137     4135       -2     
=============================================
  Files            743      743              
  Lines          26517    26518       +1     
  Branches        3294     3294              
=============================================
- Hits           12859    12846      -13     
- Misses         12262    12272      +10     
- Partials        1396     1400       +4     
Impacted Files Coverage Δ
...seata/core/exception/AbstractExceptionHandler.java 0.00% <0.00%> (ø)
...java/io/seata/rm/datasource/DataSourceManager.java 24.39% <0.00%> (ø)
...a/io/seata/rm/datasource/xa/ResourceManagerXA.java 20.89% <0.00%> (ø)
...o/seata/server/coordinator/DefaultCoordinator.java 44.26% <0.00%> (ø)
...in/java/io/seata/server/session/SessionHolder.java 60.43% <0.00%> (ø)
.../java/io/seata/server/coordinator/DefaultCore.java 45.29% <100.00%> (+0.32%) ⬆️
...n/src/main/java/io/seata/common/util/IdWorker.java 77.08% <0.00%> (-6.25%) ⬇️
...rage/redis/store/RedisTransactionStoreManager.java 72.06% <0.00%> (-2.09%) ⬇️
...torage/file/store/FileTransactionStoreManager.java 55.62% <0.00%> (-0.65%) ⬇️
...erver/storage/file/session/FileSessionManager.java 47.77% <0.00%> (-0.64%) ⬇️
... and 1 more

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 changed the title optimize log message level optimize: log message level Dec 28, 2022
Copy link
Member

@slievrly slievrly 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 merged commit e423b38 into apache:develop Dec 29, 2022
@wangliang181230 wangliang181230 added this to the 1.7.0 milestone Feb 7, 2023
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

部分日志打印级别不准确,需要梳理更正

5 participants