opensearch-project / performance-analyzer

📈 Get detailed performance metrics from your cluster independently of the Java Virtual Machine (JVM)
https://opensearch.org/docs/latest/monitoring-plugins/pa/index/
Apache License 2.0
36 stars 66 forks source link

Adds the listener for resource utilization metrics #687

Closed Gaganjuneja closed 2 months ago

Gaganjuneja commented 2 months ago

Adds the listener for resource utilization metrics.

  1. RTFPerformanceAnalyzerSearchListener - Emits the CPU_Utilization and Heap_used metrics for task level query shard operations (both query phase and the fetch phase).
  2. RTFPerformanceAnalyzerTransportInterceptor - Emits the CPU_Utilization metric for BulkShardRequest.

Is your feature request related to a problem? Please provide an existing Issue # , or describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you are proposing A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

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.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 72.54902% with 70 lines in your changes missing coverage. Please review.

Project coverage is 70.95%. Comparing base (9cb092f) to head (6899623). Report is 1 commits behind head on main.

Files Patch % Lines
...listener/RTFPerformanceAnalyzerSearchListener.java 64.39% 42 Missing and 5 partials :warning:
...RTFPerformanceAnalyzerTransportRequestHandler.java 65.21% 11 Missing and 5 partials :warning:
...performanceanalyzer/PerformanceAnalyzerPlugin.java 33.33% 2 Missing :warning:
...analyzer/config/PerformanceAnalyzerController.java 0.00% 1 Missing :warning:
...er/listener/PerformanceAnalyzerSearchListener.java 80.00% 0 Missing and 1 partial :warning:
...rt/PerformanceAnalyzerTransportRequestHandler.java 80.00% 0 Missing and 1 partial :warning:
...nsport/RTFPerformanceAnalyzerTransportChannel.java 97.91% 0 Missing and 1 partial :warning:
...rt/RTFPerformanceAnalyzerTransportInterceptor.java 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #687 +/- ## ============================================ + Coverage 70.88% 70.95% +0.07% - Complexity 421 478 +57 ============================================ Files 49 53 +4 Lines 3125 3377 +252 Branches 194 217 +23 ============================================ + Hits 2215 2396 +181 - Misses 785 844 +59 - Partials 125 137 +12 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.