Skip to content

[ISSUE #9816] Fix concurrent modify opentelemetry record in calculate consumer lag#9817

Merged
lollipopjin merged 1 commit intoapache:developfrom
lizhimins:zhimin/issue-9816
Nov 7, 2025
Merged

[ISSUE #9816] Fix concurrent modify opentelemetry record in calculate consumer lag#9817
lollipopjin merged 1 commit intoapache:developfrom
lizhimins:zhimin/issue-9816

Conversation

@lizhimins
Copy link
Copy Markdown
Member

Which Issue(s) This PR Fixes

Fixes #9816

Brief Description

How Did You Test This Change?

…culate consumer lag

Change-Id: I73109061cab5ab98fea5c1f8d23fd8bd159e2406
Signed-off-by: terrance.lzm <terrance.lzm@alibaba-inc.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.28%. Comparing base (440341b) to head (c351d72).

Files with missing lines Patch % Lines
.../rocketmq/broker/metrics/BrokerMetricsManager.java 0.00% 5 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9817      +/-   ##
=============================================
- Coverage      48.38%   48.28%   -0.10%     
+ Complexity     12254    12226      -28     
=============================================
  Files           1314     1314              
  Lines          93671    93675       +4     
  Branches       12013    12013              
=============================================
- Hits           45319    45230      -89     
- Misses         42764    42836      +72     
- Partials        5588     5609      +21     

☔ 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.

Copy link
Copy Markdown
Contributor

@lollipopjin lollipopjin left a comment

Choose a reason for hiding this comment

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

LGTM

@lollipopjin lollipopjin merged commit 736d2a8 into apache:develop Nov 7, 2025
13 of 14 checks passed
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.

[Bug] Fix concurrent modify opentelemetry record in calculate consumer lag

3 participants