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

[#9315] Add order in collected metric info, metric host info #9316

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

donghun-cho
Copy link
Contributor

@donghun-cho donghun-cho commented Oct 24, 2022

sort metric host info
sort collected metric in telegraf-metric.yml order

@codecov
Copy link

codecov bot commented Oct 24, 2022

Codecov Report

Merging #9316 (4f56548) into master (5282b05) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #9316      +/-   ##
============================================
- Coverage     39.45%   39.42%   -0.03%     
+ Complexity    11771    11764       -7     
============================================
  Files          3400     3400              
  Lines         91118    91129      +11     
  Branches      10116    10117       +1     
============================================
- Hits          35951    35930      -21     
- Misses        52067    52094      +27     
- Partials       3100     3105       +5     
Impacted Files Coverage Δ
...c/web/service/SystemMetricHostInfoServiceImpl.java 0.00% <0.00%> (ø)
.../web/service/YMLSystemMetricBasicGroupManager.java 0.00% <0.00%> (ø)
...rp/pinpoint/rpc/packet/ControlHandshakePacket.java 44.82% <0.00%> (-24.14%) ⬇️
...rc/main/java/com/navercorp/pinpoint/test/Item.java 68.51% <0.00%> (-7.41%) ⬇️
.../pinpoint/rpc/client/PinpointClientHandshaker.java 59.66% <0.00%> (-2.53%) ⬇️
...ollector/cluster/zookeeper/ZookeeperJobWorker.java 78.07% <0.00%> (-1.61%) ⬇️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 79.83% <0.00%> (-1.24%) ⬇️
...pinpoint/profiler/monitor/DeadlockMonitorTask.java 20.45% <0.00%> (-1.14%) ⬇️
...ercorp/pinpoint/common/server/util/time/Range.java 56.14% <0.00%> (ø)

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

@donghun-cho donghun-cho changed the title [#9315] sort collected metric info [#9315] Add order in collected metric info, metric host info Oct 26, 2022
Copy link
Member

@minwoo-jung minwoo-jung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good!

@donghun-cho donghun-cho merged commit 51c40e4 into pinpoint-apm:master Oct 28, 2022
@donghun-cho donghun-cho deleted the metric_sort branch December 13, 2022 01:53
@emeroad emeroad added this to the 2.5.0 milestone Dec 26, 2022
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.

None yet

3 participants