Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[3.3] Optimized EagerThreadPoolExecutorTest by decreasing maximumPoolSize #15205

Open
wants to merge 1 commit into
base: 3.3
Choose a base branch
from

Conversation

zrlw
Copy link
Contributor

@zrlw zrlw commented Mar 4, 2025

What is the purpose of the change?

try to fix #15204

Checklist

  • Make sure there is a GitHub_issue field for the change.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit-test to verify your logic correction. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
  • Make sure gitHub actions can pass. Why the workflow is failing and how to fix it?

@codecov-commenter
Copy link

codecov-commenter commented Mar 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.77%. Comparing base (3771339) to head (f91c9b7).

Additional details and impacted files
@@             Coverage Diff              @@
##                3.3   #15205      +/-   ##
============================================
- Coverage     60.78%   60.77%   -0.01%     
- Complexity    10888    10895       +7     
============================================
  Files          1885     1885              
  Lines         86071    86071              
  Branches      12892    12892              
============================================
- Hits          52315    52310       -5     
- Misses        28308    28315       +7     
+ Partials       5448     5446       -2     
Flag Coverage Δ
integration-tests 33.15% <ø> (+0.01%) ⬆️
samples-tests 29.20% <ø> (+0.02%) ⬆️
unit-tests 58.90% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] the waiting time of EagerThreadPoolExecutorTest#testEagerThreadPoolFast is insufficient
2 participants