On Mon, 31 Aug 2026 08:43:50 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).
>
> Volkan Yazici has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Copy while accessing the `idleConnectionTimeoutEvent`, which is `volatile`

src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java line 
683:

> 681:         assert stateLock.isHeldByCurrentThread();
> 682:         return streams.isEmpty() &&
> 683:                 numReservedServerStreams == 0 &&

I would ignore `numReservedServerStreams` here.

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

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

Reply via email to