Skip to content
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

optimize: the thread pool size of branchResultMessageExecutor #4670

Merged
merged 2 commits into from Jun 8, 2022
Merged

optimize: the thread pool size of branchResultMessageExecutor #4670

merged 2 commits into from Jun 8, 2022

Conversation

tuwenlin
Copy link
Contributor

@tuwenlin tuwenlin commented Jun 8, 2022

branchResultMessageExecutor只用来处理分支事务响应
ServerOnResponseProcessor
coreSize和maxSize都改为2倍核心线程数

@tuwenlin tuwenlin changed the title reduce branchResultMessageExecutor core size optimize the thread pool size of branchResultMessageExecutor Jun 8, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #4670 (7236e0a) into develop (a43beb4) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #4670      +/-   ##
=============================================
- Coverage      48.64%   48.61%   -0.04%     
+ Complexity      4044     4042       -2     
=============================================
  Files            734      734              
  Lines          25581    25583       +2     
  Branches        3157     3157              
=============================================
- Hits           12444    12437       -7     
- Misses         11802    11808       +6     
- Partials        1335     1338       +3     
Impacted Files Coverage Δ
...ava/io/seata/core/rpc/netty/NettyServerConfig.java 0.00% <0.00%> (ø)
...n/src/main/java/io/seata/common/util/IdWorker.java 77.08% <0.00%> (-6.25%) ⬇️
...very/registry/zk/ZookeeperRegisterServiceImpl.java 61.02% <0.00%> (-0.74%) ⬇️
...torage/file/store/FileTransactionStoreManager.java 56.12% <0.00%> (-0.65%) ⬇️
...erver/storage/file/session/FileSessionManager.java 49.04% <0.00%> (-0.64%) ⬇️

@funky-eyes funky-eyes changed the title optimize the thread pool size of branchResultMessageExecutor optimize: the thread pool size of branchResultMessageExecutor Jun 8, 2022
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

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

@slievrly slievrly merged commit 5729a38 into apache:develop Jun 8, 2022
liuqiufeng pushed a commit to liuqiufeng/seata that referenced this pull request Jun 11, 2022
@slievrly slievrly added this to the 1.5.2 milestone Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants