On Mon, 24 Oct 2022 17:22:21 GMT, Daniel Fuchs <dfu...@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
