Support `prate` and `pdelta` - Githubissues

newrelic / nri-jmx

New Relic Infrastructure JMX Integration

Apache License 2.0

6 stars 16 forks source link

Support `prate` and `pdelta` #54

Closed mariomac closed 4 years ago

mariomac commented 4 years ago

Description of the changes

Supported new metric_type values:
- pdelta: like delta but only reports positive values (returning 0 if the accounted value is negative)
- prate: like rate but only reports positive values (returning 0 if the accounted value is negative)

PR Review Checklist

Author

[ ] the PR should focus on a single subject. Change only relevant files to the problem you’re working on Clean and format the code
[ ] add unit tests for your changes and make sure all unit tests are passing
[ ] add a risk label after carefully considering the "blast radius" of your changes
[ ] describe the intent of your changes in the description. don't just rewrite your code in prose
[ ] assign at least one reviewer
[ ] address the feedback

Reviewer

[ ] review code for readability
[ ] verify that high risk behavior changes are well tested
[ ] check license for any new external dependency
[ ] ask questions about anything that isn't clear and obvious
[ ] approve the PR when you consider it's good to merge