Thank you, Chris, JC, and Gary, for reviewing this change!
Best regards,
Daniil
On 6/7/19, 12:31 PM, "Chris Plummer" <chris.plum...@oracle.com> wrote:
Looks good.
Chris
On 6/7/19 11:47 AM, Daniil Titov wrote:
> Please review the change that fixes an intermittent failure of the test
when it is run with Graal on.
>
> The test starts a debuggee and sets the method entry breakpoint for
tc02x004aClass1 class that has only constructor defined.
> The debuggee starts 3 threads and each of them creates a new instance of
tc02x004aClass1 class. The test waits for receiving
> 3 method entry events (posted when the constructor of tc02x004aClass1 is
invoked) during a specific amount of time.
>
> The problem here is that with Graal on it takes more time for the test to
run and occasionally the test stops listening for events
> before all method entry events are delivered.
>
> The fix ensures that the time the test waits for receiving method entry
events is adjusted for test.timeout.factor factor.
>
> The fix also makes the test waiting till the debuggee terminates before
exiting.
>
> Webrev: http://cr.openjdk.java.net/~dtitov/8222828/webrev.01
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222828
>
> Thanks!
> -Daniil
>
>
