RFR: 6967: Add Percentage Column w.r.t 'Profiling Samples' in Thread table
Alex Macdonald
aptmac at openjdk.java.net
Fri Jun 18 18:28:34 UTC 2021
On Wed, 9 Jun 2021 09:45:23 GMT, Guru Hb <ghb at openjdk.org> wrote:
> Adding "Percentage" column in Thread table (JavaApplication View). which is helpful in knowing the ratio w.r.t "Profiling Samples"
> JMC 5.x had the Percentage Column in "Hot threads" tab.
The column looks good to me, I did have a question about whether or not the strings should be externalized or not.
application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/JavaApplicationPage.java line 210:
> 208: Messages.JavaApplicationPage_COLUMN_THREAD_DURATION_DESC);
> 209: HISTOGRAM.addPercentageColumn(PROFILING_PERCENTAGE_COL, JdkAggregators.EXECUTION_SAMPLE_COUNT, "Percentage",
> 210: "Percentage over total Profiling samples");
This looks to be the only instance in the class that uses non-externalized strings, should these be put into Messages.java?
-------------
PR: https://git.openjdk.java.net/jmc/pull/260
More information about the jmc-dev
mailing list