[GitHub] [samza] cameronlee314 opened a new pull request #1593: SAMZA-2730: Add process CPU usage metric using units of processor count instead of percentage

[GitBox <gi...@apache.org>]

cameronlee314 opened a new pull request #1593:
URL: https://github.com/apache/samza/pull/1593


   Issues: Currently, "process-cpu-usage" is only reported as a percentage of the available CPU on the whole host. In some cases, it is useful to know the actual number of processors being used. For example, external systems (like an autosizer) or application owners can directly access the current processor count instead of needing to also query the total number of processors on a host in order to calculate the processor count.
   
   Changes: Add a metric `process-cpu-usage-processors` which reports the number of processors being used.
   
   Tests: Ran a container in minikube and used `LoggingMetricsReporter` to print out the values of `process-cpu-usage` and `process-cpu-usage-processors`. Verified that these values were consistent with the total processor count from running `lscpu` inside the container (i.e. (`process-cpu-usage`/100) * "total processor count from lscpu" = `process-cpu-usage-processors`).
   
   API/usage changes: (backwards compatible) An extra metric `process-cpu-usage-processors` is now available which reports the number of processors being used.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org