RFR: 8304074: [JMX] Add an approximation of total bytes allocated on the Java heap by the JVM [v3]

Paul Hohensee phh at openjdk.org
Fri May 5 20:56:41 UTC 2023


On Fri, 5 May 2023 16:16:37 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Paul Hohensee has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8304074: [JMX] Add an approximation of total bytes allocated on the Java heap by the JVM
>
> test/jdk/com/sun/management/ThreadMXBean/ThreadAllocatedMemory.java line 286:
> 
>> 284:     }
>> 285: 
>> 286:     private static long checkResult(Thread curThread,
> 
> There is another `checkResult` below? Should they be replaced by a single method?

No, there's only one checkResult method. I changed the result type from void to long.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13814#discussion_r1186491906


More information about the serviceability-dev mailing list