Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pulsar Rate Limiter refactoring #163

Closed
wants to merge 155 commits into from
Closed

Conversation

lhotari
Copy link
Owner

@lhotari lhotari commented Nov 27, 2023

@lhotari lhotari changed the title lh rate limiter refactoring Pulsar Rate Limiter refactoring Nov 27, 2023
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Merging #163 (cdda149) into master (f51e7ef) will increase coverage by 0.57%.
Report is 124 commits behind head on master.
The diff coverage is 84.16%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #163      +/-   ##
============================================
+ Coverage     72.80%   73.38%   +0.57%     
- Complexity    32381    32674     +293     
============================================
  Files          1888     1891       +3     
  Lines        140298   140494     +196     
  Branches      15420    15455      +35     
============================================
+ Hits         102144   103099     +955     
+ Misses        30063    29324     -739     
+ Partials       8091     8071      -20     
Flag Coverage Δ
inttests 24.22% <32.00%> (?)
systests 24.79% <29.33%> (+0.12%) ⬆️
unittests 72.68% <83.91%> (+0.49%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...a/org/apache/bookkeeper/mledger/ManagedLedger.java 0.00% <ø> (ø)
...che/bookkeeper/mledger/impl/ManagedCursorImpl.java 79.35% <100.00%> (+0.28%) ⬆️
...ookie/rackawareness/BookieRackAffinityMapping.java 80.85% <100.00%> (+0.13%) ⬆️
...org/apache/pulsar/broker/ServiceConfiguration.java 99.38% <100.00%> (+<0.01%) ⬆️
...authentication/AuthenticationDataSubscription.java 64.28% <100.00%> (+2.74%) ⬆️
...ar/broker/authorization/AuthorizationProvider.java 18.51% <ø> (+9.25%) ⬆️
...ache/pulsar/broker/resources/ClusterResources.java 73.80% <100.00%> (+3.14%) ⬆️
.../pulsar/broker/resources/LoadBalanceResources.java 100.00% <100.00%> (ø)
...pache/pulsar/broker/resources/PulsarResources.java 84.78% <100.00%> (ø)
...ache/pulsar/broker/ManagedLedgerClientFactory.java 84.14% <100.00%> (+1.02%) ⬆️
... and 98 more

... and 147 files with indirect coverage changes

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.

1 participant