On Wed, 9 Dec 2020 15:37:42 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

> Hi, 
> 
> Please find here a changeset that fixes the infrequent (but annoying) test 
> failures
> caused by unexpected ConnectionException "Connection timed out: no further 
> information"
> which have been observed to occur on some platforms.
> 
> Tests are updated to allow the test server to handle requests concurrently.
> PlainHttpConnection is updated to retry connection once if 
> chan::finishConnect fails
> early with ConnectionException and the connection timeout has not expired.

src/java.net.http/share/classes/jdk/internal/net/http/PlainHttpConnection.java 
line 205:

> 203:         }
> 204:         return cf.handle((r,t) -> checkRetryConnect(r, t,exchange))
> 205:                 .thenCompose(Function.identity());

So this changes behaviour so that a single subsequent additional TCP connection 
may be tried for all cases where the connection fails AND there is remaining 
connection deadline. Ok.

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

PR: https://git.openjdk.java.net/jdk/pull/1716

Reply via email to