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 233: > 231: * Define the server side of the test. > 232: * > 233: * If the server prematurely exits, serverReady will be countdown Sounds a bit strange, maybe 'will be counted down' or 'will be decremented' ? test/jdk/javax/net/ssl/Stapling/HttpsUrlConnClient.java line 320: > 318: * Define the client side of the test. > 319: * > 320: * If the server prematurely exits, serverReady will be countdown Sounds a bit strange, maybe 'will be counted down' or 'will be decremented' ? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28784#discussion_r2619903098 PR Review Comment: https://git.openjdk.org/jdk/pull/28784#discussion_r2619904360
