-
Notifications
You must be signed in to change notification settings - Fork 979
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
JvmHeapPressureMetrics have incorrect base unit #3236
Comments
We can't change the values in a way that would be backward compatible. Are you saying that what we send to Datadog is in a format that is not correct? |
No, I'm saying that the value range does not match the unit. The values for that metric are 0-1, but the unit is "percent" so Datadog expects 0-100. So then when you graph these metrics, Datadog will automatically show you readings like "1% GC overhead" because of the unit it receives, and then you're like, "oh, 1%, that's not very much at all" when in fact it is 100% because the value range doesn't match the unit. And then you have to know that the value range and the unit don't match and multiply with 100 or explicitly override the unit to make it show correct things. (Unless any of this changed in the past year and a half, I can't really check right now.) |
If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed. |
Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open. |
Thanks for the clarification @fkrull and sorry the issue was closed. Thanks to @izeye's pull request, going forward the unit will not be specified. If you would like to test it out, the changes are available in 1.13.0-SNAPSHOT in the snapshot repo now before the 1.13.0-M2 pre-release. Edit: fix version numbers |
@shakuzen Typos in the versions for 1.13.0-SNAPSHOT and 1.13.0-M2. |
@izeye Thanks for catching that. I've edited my comment now. |
Describe the bug
The unit for the metrics "jvm.memory.usage.after.gc" and "jvm.gc.overhead" in
JvmHeapPressureMetrics.java
is set as percent, but the values are in range [0...1]. This confuses Datadog (and possibly other registries?) which shows these metrics as e.g. "0.1%" instead of 10%.Environment
Expected behavior
The unit should match the value range so that Datadog can display it properly. Datadog seems to support a "fraction" unit that works for this range.
The text was updated successfully, but these errors were encountered: