Skip to content

Conversation

@LetLetMe
Copy link
Contributor

@LetLetMe LetLetMe commented Nov 6, 2024

Which Issue(s) This PR Fixes

Fixes #8895

image

Brief Description

How Did You Test This Change?

@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2024

Codecov Report

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

Project coverage is 47.75%. Comparing base (dd62ed0) to head (8dbb07c).
Report is 6 commits behind head on develop.

Files with missing lines Patch % Lines
...rocketmq/store/queue/RocksDBConsumeQueueStore.java 80.00% 3 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8896      +/-   ##
=============================================
+ Coverage      47.61%   47.75%   +0.13%     
- Complexity     11752    11812      +60     
=============================================
  Files           1304     1304              
  Lines          91043    91476     +433     
  Branches       11675    11832     +157     
=============================================
+ Hits           43354    43687     +333     
- Misses         42352    42427      +75     
- Partials        5337     5362      +25     

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

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] CreateFlushConsumeQueueService for loadAndStartConsumerServiceOnly

3 participants