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