On Fri, 9 May 2025 07:14:11 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
> Thank you. I have imported your PR locally and running some HTTP client tests > in the CI. Tests have not finished running - but I already see one > intermittent failure: `java/net/httpclient/RedirectTimeoutTest.java` is > timing out intermittently on windows. It would be good to flush out any such > intermittent failures before this PR is integrated. This might require > multiple runs before we can get confidence. My change of timeout factor to `0.7` is only temporal, as I said: it will be reverted to `4` before integration. Naturally, a few test cases will timeout when I do this /temporal/ change, hopefully `java/net/httpclient/RedirectTimeoutTest.java` will behave well with a timeout factor of `1` instead of `0.7`, but note that I will revert the timeout factor to `4` before integration. The whole idea of running with a timeout factor of `0.7` is to remove intermittent failures. (I had it close to 0.5 or maybe less to begin with until I found and reported CODETOOLS-7903937: JTREG uses timeout factor on socket timeout but not on KEEPALIVE) ------------- PR Comment: https://git.openjdk.org/jdk/pull/25122#issuecomment-2865685066
