On Wed, 26 Aug 2026 19:29:33 GMT, Volkan Yazici <[email protected]> wrote:

> Fix the reuse of timed out idle HTTP Client connections when the eviction 
> gets delayed.
> 
> The added test has certain assumptions to reproduce the problem. That said, 
> it consistently, almost always, fails if the associated fix is missing. Put 
> another way, the test can pass without the fix.
> 
> ---------
> - [X] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

src/java.net.http/share/classes/jdk/internal/net/http/ConnectionPool.java line 
347:

> 345:                     ChronoUnit.MILLIS);
> 346:         }
> 347:         return closelist;

I don't see why this piece of code could not be moved right after calling 
`purgeExpiredConnections` when obtaining a new deadline is required. It would 
avoid the ugly long[] parameter.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32545#discussion_r3870421211

Reply via email to