rocketmq icon indicating copy to clipboard operation
rocketmq copied to clipboard

[ISSUE #8483] Optimize unnecessary broker reverse notification (notifyConsumerIdsChanged) in broadcast mode

Open yx9o opened this issue 1 year ago • 1 comments
trafficstars

Fixes #8483 .

yx9o avatar Aug 03 '24 02:08 yx9o

Codecov Report

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

Project coverage is 46.94%. Comparing base (d3d7b28) to head (0416c9a). Report is 1 commits behind head on develop.

Files Patch % Lines
...apache/rocketmq/broker/client/ConsumerManager.java 66.66% 0 Missing and 2 partials :warning:
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8484      +/-   ##
=============================================
- Coverage      46.98%   46.94%   -0.04%     
  Complexity     11327    11327              
=============================================
  Files           1275     1275              
  Lines          89067    89088      +21     
  Branches       11456    11462       +6     
=============================================
- Hits           41844    41821      -23     
- Misses         42043    42083      +40     
- Partials        5180     5184       +4     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Aug 03 '24 06:08 codecov-commenter