(teststabilization) RFR: 8208281: java/nio/channels/AsynchronousSocketChannel/Basic.java timed out

Alan Bateman Alan.Bateman at oracle.com
Fri Dec 20 11:30:23 UTC 2019


On 20/12/2019 11:26, Daniel Fuchs wrote:
> :
>
> This fix proposes to simply accept the socket before closing
> the client. I haven't been able to reproduce the issue, so I
> have no proof that my suspicion is right.
>
> However, the fix is trivial, eliminates that possible cause
> for the failure, and doesn't change what the test is testing
> in any way, so I believe it's worth doing it.
I think this change is benign but I think it removes one line of 
investigation (my guess is that the issue is somewhere else). Can you 
use "peer" instead of "ch" to keep it consistent with the existing usage?

-Alan.


More information about the nio-dev mailing list