RFR: 8274: Add thread counts to the Java Application page
Alex Macdonald
aptmac at openjdk.org
Fri Nov 1 14:48:40 UTC 2024
On Fri, 1 Nov 2024 00:36:33 GMT, Marcus Hirt <hirt at openjdk.org> wrote:
> Added all of them, but perhaps only having active and daemon would be good enough.
application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/JavaApplicationPage.java line 266:
> 264: this::isAttributeEnabled, UnitLookup.PERCENT.quantity(0), UnitLookup.PERCENT.quantity(100))
> 265: .ifPresent(rows::add);
> 266: DataPageToolkit.buildLinesRow("Threads", "Various thread counts", allItems, true,
Do you think this might be better left as not filled? At least on Linux, the filled versions aren't transparent so overlaying a couple of these new values quickly covers up whatever is behind it. And with peak and total being the 3rd and 4th options, combining those with either of the first two and it gets hard to read.
example:
With two enabled:

Peak threads covers up the first two:

And then total threads covers everything:

-------------
PR Review Comment: https://git.openjdk.org/jmc/pull/600#discussion_r1825906721
More information about the jmc-dev
mailing list