rocketmq icon indicating copy to clipboard operation
rocketmq copied to clipboard

[ISSUE #8173] support ratelimit for topic or client ip

Open yuz10 opened this issue 1 year ago • 1 comments

Which Issue(s) This PR Fixes

Fixes #8173

Brief Description

How Did You Test This Change?

yuz10 avatar May 20 '24 17:05 yuz10

Codecov Report

Attention: Patch coverage is 28.82414% with 684 lines in your changes are missing coverage. Please review.

Project coverage is 42.71%. Comparing base (1b42515) to head (1c3f8d8). Report is 1 commits behind head on develop.

Files Patch % Lines
...q/ratelimit/provider/DefaultRatelimitProvider.java 17.24% 70 Missing and 2 partials :warning:
...ocketmq/broker/processor/AdminBrokerProcessor.java 0.00% 68 Missing :warning:
...s/command/ratelimit/UpdateRatelimitSubCommand.java 0.00% 59 Missing :warning:
...s/command/ratelimit/CreateRatelimitSubCommand.java 0.00% 58 Missing :warning:
...ols/command/ratelimit/ListRatelimitSubCommand.java 0.00% 44 Missing :warning:
...atelimit/manager/RatelimitMetadataManagerImpl.java 58.00% 26 Missing and 16 partials :warning:
...s/command/ratelimit/DeleteRatelimitSubCommand.java 0.00% 38 Missing :warning:
...e/rocketmq/broker/ratelimit/RatelimitPipeline.java 0.00% 37 Missing :warning:
...g/apache/rocketmq/client/impl/MQClientAPIImpl.java 0.00% 29 Missing :warning:
...rocketmq/remoting/protocol/body/RatelimitInfo.java 0.00% 26 Missing :warning:
... and 26 more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8174      +/-   ##
=============================================
- Coverage      42.90%   42.71%   -0.19%     
- Complexity     10370    10468      +98     
=============================================
  Files           1270     1297      +27     
  Lines          88721    89670     +949     
  Branches       11405    11512     +107     
=============================================
+ Hits           38062    38301     +239     
- Misses         45954    46613     +659     
- Partials        4705     4756      +51     

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

codecov-commenter avatar May 21 '24 03:05 codecov-commenter