Skip to content

Conversation

@donghun-cho
Copy link
Contributor

throttled logger applied to StreamExecutorServerInterceptor

@CLAassistant
Copy link

CLAassistant commented Apr 6, 2022

CLA assistant check
All committers have signed the CLA.

@donghun-cho donghun-cho changed the title [#4817] apply throttled logger [#noissue] apply throttled logger Apr 6, 2022
@codecov
Copy link

codecov bot commented Apr 7, 2022

Codecov Report

Merging #8751 (965587a) into master (876d852) will increase coverage by 0.03%.
The diff coverage is 61.53%.

@@             Coverage Diff              @@
##             master    #8751      +/-   ##
============================================
+ Coverage     39.97%   40.00%   +0.03%     
- Complexity    11504    11513       +9     
============================================
  Files          3381     3381              
  Lines         89450    89460      +10     
  Branches       9963     9964       +1     
============================================
+ Hits          35757    35790      +33     
+ Misses        50630    50604      -26     
- Partials       3063     3066       +3     
Impacted Files Coverage Δ
...ervice/StreamExecutorServerInterceptorFactory.java 0.00% <0.00%> (ø)
...lient/interceptor/LoggingDiscardEventListener.java 50.00% <ø> (ø)
...r/flowcontrol/StreamExecutorServerInterceptor.java 0.00% <0.00%> (ø)
.../pinpoint/profiler/sender/grpc/GrpcDataSender.java 0.00% <ø> (ø)
...collector/grpc/config/GrpcStreamConfiguration.java 71.42% <85.71%> (+1.98%) ⬆️
...point/common/profiler/logging/ThrottledLogger.java 18.86% <100.00%> (ø)
...orp/pinpoint/profiler/sender/NioUDPDataSender.java 62.82% <0.00%> (-5.13%) ⬇️
...rc/main/java/com/navercorp/pinpoint/test/Item.java 72.22% <0.00%> (-3.71%) ⬇️
.../pinpoint/rpc/client/PinpointClientHandshaker.java 59.66% <0.00%> (-2.53%) ⬇️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 81.06% <0.00%> (+1.23%) ⬆️
... and 5 more

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 876d852...965587a. Read the comment docs.

@donghun-cho donghun-cho changed the title [#noissue] apply throttled logger [#8762] apply throttled logger Apr 8, 2022
@donghun-cho donghun-cho merged commit e1542db into pinpoint-apm:master Apr 12, 2022
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.

3 participants