On Thu, 4 Feb 2021 03:40:26 GMT, Chris Plummer <[email protected]> wrote:

> > I am not reviewer, but maybe you could add test for "parallel" in 
> > BasicJMapTest.java.
> 
> That test is for the `jmap` command, not the `GC.class_histogram` jcmd. As I 
> pointed out, there are tests in `test/hotpot/jtreg/serviceability/dcmd/gc` 
> that already test `GC.class_histogram`. That would be a better starting point.

Correct!  Thanks for point it out, and sorry for misleading the test case.

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

PR: https://git.openjdk.java.net/jdk/pull/2379

Reply via email to