RFR: 8317287: [macos14] InterJVMGetDropSuccessTest.java: Child VM: abnormal termination [v10]

songpv-imt duke at openjdk.org
Tue Jan 16 17:29:52 UTC 2024


On Tue, 16 Jan 2024 14:42:07 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> In addition to this, I propose removing `robot.delay(50);` from the for-loops, or at least reducing this value, let's say, to 5.
>
>> In addition to this, I propose removing `robot.delay(50);` from the for-loops, or at least reducing this value, let's say, to 5.
> 
> The test is unstable without a delay inside the for-loop.
> 
> `robot.delay(5)` does the trick. The test passes successfully with 50 repeats.
> 
> The above changes make the test significantly quicker.
> 
> - Original version takes **25 minutes** for 50 repeats;
> - Yesterday's version — 30 minutes;
> - Today's version — 22 minutes;
> - With `delay(5)` — **7 minutes**;
> - ~~No delay — 5 minutes (*unstable*)~~.
> 
> Thus, the modified version with `robot.delay(5)` instead of `robot.delay(50)` completes nearly twice as quick compared to the original test.

Thank you, the code has been updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16396#discussion_r1453747130


More information about the client-libs-dev mailing list