Integrated: 7506: Incorrect numeric formatting of PID by JMC
Suchita Chaturvedi
schaturvedi at openjdk.org
Fri Jun 14 05:58:23 UTC 2024
On Sat, 25 May 2024 21:25:09 GMT, Suchita Chaturvedi <schaturvedi at openjdk.org> wrote:
> This PR is to fix the formatting issue of the JVM PID on JVM information screen. JVM PID attribute is received as NUMBER from JFR and hence the JMC application is treating is as number. When the number is smaller , the PID is displayed using commas and when the number is larger, its converted to exponential format. This is a long pending bug in product.
>
> The PID (Process Id) attribute is present in JMC as part of System Process event also where the data type is correct i.e. String. So in order to maintain consistency, we have made the content type of PID attribute of JVM Information event also as String. We are converting the number value of PID to string and then storing it.
>
> We have also introduced a new aggregator PID which will contain the value of Identifier in plain text format instead of number format. This new aggregator is displayed on JVM Information page.
This pull request has now been integrated.
Changeset: 992333a5
Author: Suchita Chaturvedi <schaturvedi at openjdk.org>
URL: https://git.openjdk.org/jmc/commit/992333a5259cfbcf0db1a08cc34936d189c962e3
Stats: 73 lines in 4 files changed: 66 ins; 0 del; 7 mod
7506: Incorrect numeric formatting of PID by JMC
Co-authored-by: Marcus Hirt <hirt at openjdk.org>
Reviewed-by: bdutheil, vpurnam, hirt
-------------
PR: https://git.openjdk.org/jmc/pull/565
More information about the jmc-dev
mailing list