On Mon, 15 Dec 2025 13:01:29 GMT, Arno Zeller <[email protected]> wrote:
>> The test javax/net/ssl/Stapling/HttpsUrlConnClient.java sometimes hangs and
>> run into timeouts. By adding some more tracing it showed that it can happen
>> on machines with high load that the client thread throws a "Server not ready
>> yet" RuntimeException because it only waits for 5 seconds for the server
>> thread to come available.
>> I added some more output in case of errors to better see the issue and use
>> the timeout factor to modify the maximum wait time.
>
> Arno Zeller has updated the pull request with a new target base due to a
> merge or a rebase. The incremental webrev excludes the unrelated changes
> brought in by the merge/rebase. The pull request contains four additional
> commits since the last revision:
>
> - Use CountDownLatch
> - Merge remote-tracking branch 'upstream/master' into JDK-8371559
> - Merge branch 'openjdk:master' into JDK-8371559
> - Use timeout factor when waiting for server ready
test/jdk/javax/net/ssl/Stapling/HttpsUrlConnClient.java line 475:
> 473: * Release the client, if not active already...
> 474: */
> 475: System.err.println("Server died: " + e);
Maybe do e.printStackTrace(System.err); too ? but leave it to you , it is
now at least better than before ....
(same for some other exception handling coding you changed)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28784#discussion_r2619918329
