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