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

[#9086] Lightened agentCountJob #9096

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

smilu97
Copy link
Contributor

@smilu97 smilu97 commented Aug 4, 2022

  • AgentCountJob only counts agents, not doing others
  • GET /selectLatestAgentCount is added

@smilu97 smilu97 self-assigned this Aug 4, 2022
@codecov
Copy link

codecov bot commented Aug 4, 2022

Codecov Report

Merging #9096 (3420689) into master (012fb56) will decrease coverage by 0.00%.
The diff coverage is 13.11%.

@@             Coverage Diff              @@
##             master    #9096      +/-   ##
============================================
- Coverage     39.52%   39.51%   -0.01%     
  Complexity    11664    11664              
============================================
  Files          3351     3351              
  Lines         90371    90406      +35     
  Branches      10059    10061       +2     
============================================
+ Hits          35718    35724       +6     
- Misses        51612    51638      +26     
- Partials       3041     3044       +3     
Impacted Files Coverage Δ
...ercorp/pinpoint/batch/job/AgentCountProcessor.java 0.00% <0.00%> (ø)
...navercorp/pinpoint/batch/job/AgentCountReader.java 0.00% <0.00%> (ø)
...navercorp/pinpoint/batch/job/AgentCountWriter.java 0.00% <0.00%> (ø)
...oint/web/controller/AgentStatisticsController.java 0.00% <0.00%> (ø)
...inpoint/web/dao/mysql/MysqlAgentStatisticsDao.java 0.00% <0.00%> (ø)
...npoint/web/service/AgentStatisticsServiceImpl.java 0.00% <0.00%> (ø)
...point/web/dao/memory/MemoryAgentStatisticsDao.java 88.88% <72.72%> (-11.12%) ⬇️
...vercorp/pinpoint/rpc/packet/ServerClosePacket.java 47.82% <0.00%> (-17.40%) ⬇️
.../navercorp/pinpoint/rpc/client/RequestManager.java 66.66% <0.00%> (-7.58%) ⬇️
...rp/pinpoint/rpc/server/PinpointServerAcceptor.java 84.10% <0.00%> (-4.64%) ⬇️
... and 7 more

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

@smilu97 smilu97 force-pushed the #9086_lighten_counting_agents branch 5 times, most recently from 58d6546 to c42ff7c Compare August 9, 2022 02:16
@smilu97 smilu97 force-pushed the #9086_lighten_counting_agents branch from c42ff7c to 3420689 Compare August 9, 2022 02:20
@smilu97 smilu97 merged commit 7facdf2 into pinpoint-apm:master Aug 10, 2022
@smilu97 smilu97 deleted the #9086_lighten_counting_agents branch August 10, 2022 05:37
@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.

2 participants