On Wed, 20 Sep 2023 05:32:07 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Test start failing because of unexpected VM logging.
>> The fix is to search expected string ignoring any unknown output from forked 
>> process.
>> Tested with tier1 and by running test with various options used in CI.
>
> test/hotspot/jtreg/serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java 
> line 95:
> 
>> 93:                 } else {
>> 94:                     System.out.println("Ignoring: " + pidstring);
>> 95:                 }
> 
> Do we really want this in the test output?

It might be helpful to ensure that we don't mess with matching PID line if test 
fails.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15833#discussion_r1331032496

Reply via email to