jaegertracing / legacy-client-java

Legacy com.uber.jaeger java client
Apache License 2.0
5 stars 6 forks source link

Add ability to update rate limiter #17

Open black-adder opened 6 years ago

black-adder commented 6 years ago

Signed-off-by: Won Jun Jang wjang@uber.com

Which problem is this PR solving?

cf. jaegertracing/jaeger-client-go#320

codecov[bot] commented 6 years ago

Codecov Report

Merging #17 into master will increase coverage by 0.19%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #17      +/-   ##
============================================
+ Coverage     84.77%   84.96%   +0.19%     
- Complexity      608      618      +10     
============================================
  Files            94       94              
  Lines          2410     2428      +18     
  Branches        271      271              
============================================
+ Hits           2043     2063      +20     
+ Misses          276      275       -1     
+ Partials         91       90       -1
Impacted Files Coverage Δ Complexity Δ
...r/jaeger/samplers/GuaranteedThroughputSampler.java 88.88% <100%> (ø) 7 <0> (ø) :arrow_down:
.../com/uber/jaeger/samplers/RateLimitingSampler.java 82.6% <100%> (+15.94%) 10 <5> (+4) :arrow_up:
...c/main/java/com/uber/jaeger/utils/RateLimiter.java 100% <100%> (ø) 8 <6> (+3) :arrow_up:
.../uber/jaeger/samplers/RemoteControlledSampler.java 86.53% <100%> (+0.67%) 23 <3> (+3) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7a82dd0...0a5f30a. Read the comment docs.