Skip to content

[ISSUE #9530]opti: Reduce unnecessary memory allocation in adaptiveBackOffSpinLock - #9530

Merged
tianliuliu merged 1 commit into
apache:developfrom
3424672656:opti_lock
Jul 10, 2025
Merged

[ISSUE #9530]opti: Reduce unnecessary memory allocation in adaptiveBackOffSpinLock#9530
tianliuliu merged 1 commit into
apache:developfrom
3424672656:opti_lock

Conversation

@3424672656

Copy link
Copy Markdown
Contributor

Which Issue(s) This PR Fixes

Fixes #9529

Brief Description

How Did You Test This Change?

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.92%. Comparing base (9337904) to head (db9b49b).

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9530      +/-   ##
=============================================
- Coverage      48.00%   47.92%   -0.08%     
+ Complexity     12026    12012      -14     
=============================================
  Files           1312     1312              
  Lines          92696    92696              
  Branches       11856    11856              
=============================================
- Hits           44496    44425      -71     
- Misses         42687    42755      +68     
- Partials        5513     5516       +3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@3424672656 3424672656 changed the title [ISSUE #9530]opti: opti thread count in adaptiveBackOffSpinLock [ISSUE #9530]opti: Reduce unnecessary memory allocation in adaptiveBackOffSpinLock Jul 9, 2025
@tianliuliu
tianliuliu self-requested a review July 10, 2025 02:17
this.threadTable = new ArrayList<>(2);
this.threadTable.add(new ConcurrentHashMap<>());
this.threadTable.add(new ConcurrentHashMap<>());
this.threadTable.add(ConcurrentHashMap.newKeySet());

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tianliuliu
tianliuliu merged commit 1a747d1 into apache:develop Jul 10, 2025
13 of 19 checks passed
hechen0 pushed a commit to hechen0/rocketmq that referenced this pull request Aug 18, 2025
Co-authored-by: hqbfzwang <hqbfzwang@tencent.com>
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.

[Enhancement] Reduce unnecessary memory allocation

3 participants