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

[ISSUE #9184] Optimize QueueLockManager#tryLock method in Pop #9185

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

Conversation

mxsm
Copy link
Member

@mxsm mxsm commented Feb 16, 2025

Which Issue(s) This PR Fixes

Fixes #9184

Brief Description

How Did You Test This Change?

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 48.08%. Comparing base (295821b) to head (858702b).

Files with missing lines Patch % Lines
...rocketmq/broker/processor/PopMessageProcessor.java 80.00% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9185      +/-   ##
=============================================
- Coverage      48.13%   48.08%   -0.05%     
+ Complexity     12104    12092      -12     
=============================================
  Files           1321     1321              
  Lines          93033    93026       -7     
  Branches       11924    11920       -4     
=============================================
- Hits           44777    44734      -43     
- Misses         42737    42765      +28     
- Partials        5519     5527       +8     

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

@lizhimins lizhimins changed the title [ISSUE #9184]Optimize QueueLockManager#tryLock method [ISSUE #9184] Optimize QueueLockManager#tryLock method in Pop Feb 17, 2025
# 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.

[Enhancement] Optimize QueueLockManager#tryLock method
5 participants