On Fri, 4 Apr 2025 16:36:38 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

>> Good point - our default idle timeout is 30s IIRC. There would need to be a 
>> pause of 30s between the two calls to `client.send` however. Not impossible 
>> but unlikely. We could either pass a higher timeout (with e.g. 
>> `-Djdk.httpclient.keepalive.timeout=120`) preemptively, or wait until we see 
>> the test fail...
>
> Waiting for the test to fail due to this reason and then adjusting the 
> timeout is fine with me.

On the plus side - if we add the `-Djdk.httpclient.keepalive.timeout=120` now 
we might not have to deal with it later, and maybe it will give us a clue if 
the test ever fails due to this.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2029133030

Reply via email to