RFR: 8288717: Add a means to close idle connections in HTTP/2 connection pool [v4]

Conor Cleary ccleary at openjdk.org
Tue Oct 25 08:19:47 UTC 2022


On Mon, 24 Oct 2022 17:22:21 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Conor Cleary has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8288717: Updated test and property to use seconds
>
> test/jdk/java/net/httpclient/http2/IdleConnectionTimeoutTest.java line 95:
> 
>> 93:             if (timeoutVal.equals("1")) {
>> 94:                 hreq = HttpRequest.newBuilder(timeoutUri).version(HTTP_2).GET().build();
>> 95:                 sleepTime = 8000;
> 
> Do we really need to sleep for 8 seconds? would 4 work too?

We could maybe even go further and maybe use 1 and 2 second property values in the test runs? Then sleep values of 2000ms and 1000ms respectively.

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

PR: https://git.openjdk.org/jdk/pull/10183


More information about the net-dev mailing list