RFR: 8358496: Concurrent reading from Socket with timeout executes sequentially [v2]

Alan Bateman alanb at openjdk.org
Tue Jun 3 16:10:38 UTC 2025


On Tue, 3 Jun 2025 15:20:48 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Alan Bateman has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Make test comment cleaner
>
> test/jdk/java/net/Socket/Timeouts.java line 249:
> 
>> 247:                         () -> s.getInputStream().read());
>> 248:                 int timeout = s.getSoTimeout();
>> 249:                 checkDuration(startMillis, timeout-100, timeout+20_000);
> 
> Maybe it would be worth noting here that if the bug isn't fixed, the last thread to call read() would have to wait for 200 seconds, which largely exceeds 22s?

Okay, I can make the test method description a bit clearer on this point.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25614#discussion_r2124337179


More information about the net-dev mailing list