Skip to content

[ISSUE #9605] Fixed a potential resource leak#9606

Merged
lizhimins merged 1 commit intoapache:developfrom
yx9o:dev-9605
Aug 9, 2025
Merged

[ISSUE #9605] Fixed a potential resource leak#9606
lizhimins merged 1 commit intoapache:developfrom
yx9o:dev-9605

Conversation

@yx9o
Copy link
Copy Markdown
Contributor

@yx9o yx9o commented Aug 8, 2025

Fixes #9605 .

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 8, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 48.03%. Comparing base (9609722) to head (989d240).
⚠️ Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
...a/org/apache/rocketmq/broker/BrokerController.java 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9606      +/-   ##
=============================================
- Coverage      48.15%   48.03%   -0.13%     
+ Complexity     12085    12047      -38     
=============================================
  Files           1313     1313              
  Lines          92791    92793       +2     
  Branches       11866    11867       +1     
=============================================
- Hits           44681    44569     -112     
- Misses         42617    42703      +86     
- Partials        5493     5521      +28     

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

@lizhimins lizhimins merged commit 6d4fb83 into apache:develop Aug 9, 2025
10 of 13 checks passed
@yx9o yx9o deleted the dev-9605 branch August 9, 2025 11:52
hechen0 pushed a commit to hechen0/rocketmq that referenced this pull request Aug 18, 2025
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] There is a risk of resource leakage

3 participants