RFR: 8145408: "com/sun/jdi/BreakpointWithFullGC.sh: Required output "Full GC" not found"

[Alexander Kulyakhtin]

Daniel,

Thank you very much for the review. I'm going to add the comment as you have suggested.

Best regards,
Alexander

----- Original Message -----
From: daniel.daugherty at oracle.com
To: alexander.kulyakhtin at oracle.com, serviceability-dev at openjdk.java.net
Sent: Wednesday, December 16, 2015 5:58:03 PM GMT +03:00 Iraq
Subject: Re: RFR: 8145408: "com/sun/jdi/BreakpointWithFullGC.sh: Required output "Full GC" not found"

You might want to include a comment like:

# Prior to JDK9-BNN, the pattern was 'Full GC'

where NN is the promoted build where the format changed.
This will help anyone backporting test fixes to older releases.

Dan

On 12/16/15 5:05 AM, Alexander Kulyakhtin wrote:
> Hi,
>
> Could you, please, review this small test-only fix:
>
> CR: https://bugs.openjdk.java.net/browse/JDK-8145408 "com/sun/jdi/BreakpointWithFullGC.sh: Required output "Full GC" not found"
> Webrev: http://cr.openjdk.java.net/~akulyakh/8145408/test/com/sun/jdi/BreakpointWithFullGC.sh.udiff.html
>
> The test, among other things, verifies that an indication of a Full GC event is present in the output.
>
> Recently, such an indication has been changed from similar to:
> [Full GC (System.gc())  2663K->2663K(31744K), 0.0034321 secs]
>
> to similar to:
> [3.326s][info   ][gc] GC(10) Pause Full (System.gc()) 11M->3M(12M) (3.312s, 3.326s) 14.405ms
>
> We are changing the matcher correspondingly, also making it more precise.
>
> Best regards,
> Alexander