RFR: 7506: Incorrect numeric formatting of PID by JMC [v2]

Suchita Chaturvedi schaturvedi at openjdk.org
Mon Mar 25 05:18:26 UTC 2024


On Mon, 25 Mar 2024 05:05:39 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. 
>> 
>> As part of this PR we have introduced a new aggregator JVM_PID_ID which will contain the value of Identifier in plain text format instead of number format. This new aggregator is displayed on JVM Information page so that formatting is not applied as per NUMBER formatting rules.
>> 
>> JVM PID before the change:
>> <img width="359" alt="image" src="https://github.com/openjdk/jmc/assets/11155712/2e2a238c-8160-4bd1-8285-fb2a7a4fb85c">
>> 
>> JVM PID after the change:
>> <img width="355" alt="image" src="https://github.com/openjdk/jmc/assets/11155712/92668a9d-6053-4fa1-bcd0-7face596bc15">
>> 
>> The issue is more prominent when the identifier value is converted to exponential.
>
> Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fixed the pid formatting issue on Event Browser and Properties section also

For the JVM PID 2177:
![image](https://github.com/openjdk/jmc/assets/11155712/1a7ee492-a970-4c1c-826d-93b36be297c0)

**Before the Change:**
JVM Internal Page
<img width="960" alt="Screenshot 2024-03-25 103611" src="https://github.com/openjdk/jmc/assets/11155712/6a530881-5e5b-43dc-b4c7-618da7021fb9">

Event Browser Page
<img width="959" alt="Screenshot 2024-03-25 104001" src="https://github.com/openjdk/jmc/assets/11155712/22b17c1a-472a-4ce8-a0ad-c5b851dba27e">

**After the Change:**
JVM Internal Page
<img width="960" alt="Screenshot 2024-03-25 104207" src="https://github.com/openjdk/jmc/assets/11155712/81ae8a40-ef82-43f7-9fa8-198ceb893569">

Event Browser Page
<img width="958" alt="Screenshot 2024-03-25 104343" src="https://github.com/openjdk/jmc/assets/11155712/bcfa8a86-3ecd-48b5-bacd-ad35ad30448c">

-------------

PR Comment: https://git.openjdk.org/jmc/pull/557#issuecomment-2017233921


More information about the jmc-dev mailing list