On Fri, 28 Jun 2024 18:14:59 GMT, Kevin Walls <kev...@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

Reply via email to