On Tue, 3 Jun 2025 15:20:48 GMT, Daniel Fuchs <dfu...@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
