RFR: 8326498: java.net.http.HttpClient connection leak using http/2 [v2]

Jaikiran Pai jpai at openjdk.org
Mon Nov 17 12:28:37 UTC 2025


On Wed, 12 Nov 2025 10:50:01 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - mark jdk.internal.net.http.Http2Connection as Closable
>>  - reduce number of concurrent requests
>
> test/jdk/java/net/httpclient/http2/BurstyRequestsTest.java line 128:
> 
>> 126: 
>> 127:     /*
>> 128:      * Issues a burst of 100 HTTP/2 requests to the same server (host/port) and expects all of
> 
> `numRequests` is actually 20, not 100:
> 
> Suggestion:
> 
>      * Issues a burst of HTTP/2 requests to the same server (host/port) and expects all of

Good catch. This was a left over from a change where I reduced the count to 20. I have fixed this comment now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28233#discussion_r2533866188


More information about the net-dev mailing list