Skip to content

Conversation

@yjqg6666
Copy link
Contributor

@yjqg6666 yjqg6666 commented Nov 9, 2021

Resolve #8376.
image

@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

Merging #8377 (d51cd55) into master (ad8b288) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8377      +/-   ##
============================================
- Coverage     39.84%   39.84%   -0.01%     
+ Complexity    11407    11406       -1     
============================================
  Files          3340     3340              
  Lines         89303    89303              
  Branches       9913     9913              
============================================
- Hits          35587    35582       -5     
- Misses        50655    50659       +4     
- Partials       3061     3062       +1     
Impacted Files Coverage Δ
.../navercorp/pinpoint/rpc/client/RequestManager.java 66.66% <0.00%> (-7.58%) ⬇️
...cluster/zookeeper/ZookeeperClusterDataManager.java 50.32% <0.00%> (-3.27%) ⬇️
...vercorp/pinpoint/web/vo/ApplicationAgentsList.java 69.73% <0.00%> (-2.64%) ⬇️
...java/com/navercorp/pinpoint/rpc/DefaultFuture.java 73.39% <0.00%> (-1.84%) ⬇️
...pinpoint/profiler/monitor/DeadlockMonitorTask.java 20.93% <0.00%> (-1.17%) ⬇️
...point/rpc/client/DefaultPinpointClientHandler.java 72.18% <0.00%> (+1.76%) ⬆️
...navercorp/pinpoint/rpc/common/SocketStateCode.java 84.37% <0.00%> (+3.12%) ⬆️
...r/zookeeper/ZookeeperClusterDataManagerHelper.java 82.22% <0.00%> (+6.66%) ⬆️

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 ad8b288...d51cd55. Read the comment docs.

@yjqg6666
Copy link
Contributor Author

yjqg6666 commented Nov 9, 2021

@binDongKim PTAL.

@yjqg6666 yjqg6666 requested a review from binDongKim November 10, 2021 09:59
@binDongKim binDongKim merged commit d295d37 into pinpoint-apm:master Nov 11, 2021
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.

[FR.] Highlight error row in call tree

2 participants