jmx-dev RFR: 8327505: Test com/sun/jmx/remote/NotificationMarshalVersions/TestSerializationMismatch.java fails

[Thomas Stuefe]

On Tue, 19 Mar 2024 16:23:27 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Client.java has a fixed 30-second timeout on the CountDownLatch to wait for 10 notifications.
> 
> If it fails, you can't tell if CountDownLatch.await threw, or returned false and the app threw InterruptedException, due to the way Client.java handles these.
> 
> Seems most likely the 30 second wait expired, as we are dealing with -Xcomp failures in a debug build.  Passing runs can take a few seconds, but can be 25 seconds.
> 
> Increasing the timeout and tidying up the handling so we can see the specific reason in future.

Looks good!

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

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18381#pullrequestreview-1965321788