RFR: 8316562: serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java times out after JDK-8314829

Leonid Mesnik lmesnik at openjdk.org
Wed Sep 20 05:09:38 UTC 2023


On Wed, 20 Sep 2023 03:56:28 GMT, Leonid Mesnik <lmesnik at 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.

Something like this was my first fix. Use largeHeapScanner.nextLine() to read String and check it. However, it makes unclear why a scanner is needed for line-by-line parsing. So I decided to change it to make a more known pattern for parsing input by line.

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

PR Comment: https://git.openjdk.org/jdk/pull/15833#issuecomment-1726977519


More information about the serviceability-dev mailing list