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