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

[#9563] Dynamic loading of GrpcMetricHandler #9564

Merged
merged 1 commit into from
Dec 30, 2022

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Dec 29, 2022

No description provided.

@emeroad emeroad added this to the 2.5.1 milestone Dec 29, 2022
@emeroad emeroad linked an issue Dec 29, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Dec 29, 2022

Codecov Report

Merging #9564 (66ba238) into master (9f9e0b9) will increase coverage by 0.01%.
The diff coverage is 58.46%.

@@             Coverage Diff              @@
##             master    #9564      +/-   ##
============================================
+ Coverage     38.87%   38.89%   +0.01%     
- Complexity    11884    11895      +11     
============================================
  Files          3482     3485       +3     
  Lines         93734    93750      +16     
  Branches      10525    10523       -2     
============================================
+ Hits          36439    36460      +21     
  Misses        54133    54133              
+ Partials       3162     3157       -5     
Impacted Files Coverage Δ
...lector/handler/grpc/metric/AgentMetricHandler.java 27.27% <27.27%> (ø)
...r/handler/grpc/metric/AgentMetricBatchHandler.java 40.00% <40.00%> (ø)
...tor/handler/grpc/metric/AgentUriMetricHandler.java 88.23% <88.23%> (ø)
...collector/handler/grpc/GrpcAgentStatHandlerV2.java 100.00% <100.00%> (+59.61%) ⬆️
...ollector/cluster/zookeeper/ZookeeperJobWorker.java 75.93% <0.00%> (-1.61%) ⬇️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 81.06% <0.00%> (-0.83%) ⬇️
...n/server/bo/serializer/trace/v2/SpanDecoderV0.java 81.73% <0.00%> (+0.45%) ⬆️
...n/server/bo/serializer/trace/v2/SpanEncoderV0.java 83.42% <0.00%> (+0.53%) ⬆️
...pinpoint/profiler/monitor/DeadlockMonitorTask.java 21.59% <0.00%> (+1.13%) ⬆️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@emeroad emeroad merged commit c74b2bc into pinpoint-apm:master Dec 30, 2022
@emeroad emeroad deleted the #9563_dynamic_load branch December 30, 2022 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improved dynamic loading of GrpcMetricHandler for modularity
1 participant