-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add CPU metrics for minion purge task #12337
Conversation
...uiltin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/purge/PurgeTaskExecutor.java
Outdated
Show resolved
Hide resolved
f315457
to
7645bf7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for making the code changes!
7645bf7
to
198cafb
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #12337 +/- ##
============================================
- Coverage 61.65% 61.63% -0.02%
Complexity 1152 1152
============================================
Files 2421 2421
Lines 131872 131879 +7
Branches 20346 20346
============================================
- Hits 81308 81289 -19
- Misses 44600 44627 +27
+ Partials 5964 5963 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
198cafb
to
d94c1a7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add CPU metrics for minion purge task, i.e. threadCpuTime. Right now there is only task executing time metrics.