jmx-dev RFR: 8334777: Test javax/management/remote/mandatory/notif/NotifReconnectDeadlockTest.java failed with NullPointerException [v2]

[Kevin Walls]

On Fri, 28 Jun 2024 18:14:59 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Disable running this test with -Xcomp.
>> 
>> The NPE seen in this test is due to a timeout establishing the connection.  ServerCommunicatorAdmin hits its timeout, during an addNotificationListener call on a new connection.
>> 
>> -Xcomp causes this slowdown and the failure is reproducible.  There is no need to test this test with -Xcomp, we should exclude it as we do for various other tests.
>
> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
> 
>   comment

Yes sure, I've added a short comment, so it's clear this is intentional and we know -Xcomp causes a specific timeout.  Checked test still runs OK, and skips with -Xcomp.

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

PR Comment: https://git.openjdk.org/jdk/pull/19930#issuecomment-2197426365