On Wed, 12 Nov 2025 10:50:01 GMT, Volkan Yazici <[email protected]> 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
