On Tue, 14 Nov 2023 17:25:42 GMT, Darragh Clarke <dcla...@openjdk.org> wrote:

>> Currently the `IsAlive` test occasionally times out.
>> 
>> This PR changes the timeout from `10` to `25` which I think is an adequate 
>> increase based on the failures I've seen. Though I'd be happy to change it 
>> to another value if someone thinks `25` isn't ideal.
>
> Darragh Clarke has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   use default timeout

Seems fine. If it ever times out again then we know we have a real problem. :)

I suspect, as  Alan suggested, the `othervm` could be dropped, but I'm not 
concerned about it staying.

Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16659#pullrequestreview-1733586933

Reply via email to