RFR: 8288717: Add a means to close idle connections in HTTP/2 connection pool [v5]
Conor Cleary
ccleary at openjdk.org
Mon Nov 7 12:39:38 UTC 2022
On Thu, 3 Nov 2022 18:13:34 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: IdleConnectionTimeout can use Keep Alive or Custom Value
>
> src/java.net.http/share/classes/jdk/internal/net/http/HttpClientImpl.java line 1708:
>
>> 1706: if (s != null) {
>> 1707: long timeoutVal = Long.parseLong(s);
>> 1708: System.err.println(timeoutVal);
>
> Stray println? You could use `Log.logTrace` here to print the value.
Gah, thought I zapped all of these! Thanks for spotting. I think there is no need to log anything here, so I will just remove the println.
> src/java.net.http/share/classes/jdk/internal/net/http/HttpClientImpl.java line 1711:
>
>> 1709: if (timeoutVal >= 0) return timeoutVal;
>> 1710: }
>> 1711: } catch (NumberFormatException ignored) {}
>
> Similarly - you could use `Log.logTrace` to print the error. It could be useful for diagnostic.
Good spot, would be useful. Should result in better logging with the test too. I'll include this.
-------------
PR: https://git.openjdk.org/jdk/pull/10183
More information about the net-dev
mailing list