Adds Thread_Waited_Time and Thread_Waited_Event metrics#70
Adds Thread_Waited_Time and Thread_Waited_Event metrics#70sruti1312 merged 2 commits intoopensearch-project:mainfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## main #70 +/- ##
============================================
- Coverage 72.18% 72.00% -0.19%
+ Complexity 2948 2943 -5
============================================
Files 376 376
Lines 18680 18723 +43
Branches 1429 1432 +3
============================================
- Hits 13485 13481 -4
- Misses 4610 4655 +45
- Partials 585 587 +2
Continue to review full report at Codecov.
|
|
Thanks for adding these metrics. Can you merge from main and re-run the CI? I think the CI is failing due to a known error that is fixed in The metrics documentation page needs to be updated https://opendistro.github.io/for-elasticsearch-docs/docs/pa/reference. It can be done by modifying it in https://github.com/opensearch-project/opensearch-docs. |
…nce-analyzer-rca into thread_waited_time
Thanks for reviewing the CR @sruti1312 I am getting a 404 on the repo https://github.com/opensearch-project/opensearch-docs most prolly due to lack of permission. Will update those docs once I get permission. |
Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Adds new metrics published by the metrics API :
Thread_Waited_Time(https://docs.oracle.com/en/java/javase/14/docs/api/java.management/java/lang/management/ThreadInfo.html#getWaitedTime()) andThread_Waited_Event(https://docs.oracle.com/en/java/javase/14/docs/api/java.management/java/lang/management/ThreadInfo.html#getWaitedCount())Describe the solution you are proposing
Additional context
The new metrics support
Operationas a dimension.Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.