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

[#8993] Add apdex alarm #8995

Merged
merged 1 commit into from
Jul 1, 2022
Merged

Conversation

smilu97
Copy link
Contributor

@smilu97 smilu97 commented Jun 29, 2022

No description provided.

@codecov
Copy link

codecov bot commented Jun 29, 2022

Codecov Report

Merging #8995 (fb9ac6e) into master (0804653) will increase coverage by 0.01%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master    #8995      +/-   ##
============================================
+ Coverage     39.70%   39.71%   +0.01%     
- Complexity    11789    11797       +8     
============================================
  Files          3347     3348       +1     
  Lines         90917    90935      +18     
  Branches      10062    10062              
============================================
+ Hits          36096    36115      +19     
- Misses        51726    51727       +1     
+ Partials       3095     3093       -2     
Impacted Files Coverage Δ
.../navercorp/pinpoint/web/alarm/CheckerCategory.java 0.00% <0.00%> (ø)
...avercorp/pinpoint/batch/alarm/CheckerRegistry.java 63.63% <50.00%> (-0.65%) ⬇️
...inpoint/batch/alarm/checker/ApdexScoreChecker.java 100.00% <100.00%> (ø)
...tch/alarm/collector/ResponseTimeDataCollector.java 93.75% <100.00%> (+0.89%) ⬆️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 81.89% <0.00%> (+0.82%) ⬆️
...pinpoint/profiler/monitor/DeadlockMonitorTask.java 21.59% <0.00%> (+1.13%) ⬆️

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 0804653...fb9ac6e. Read the comment docs.

@smilu97 smilu97 requested a review from minwoo-jung June 30, 2022 09:10
@smilu97 smilu97 merged commit a8c8e20 into pinpoint-apm:master Jul 1, 2022
@smilu97 smilu97 deleted the #8993_add_apdex_alarm branch July 1, 2022 06:59
@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