On Tue, 23 Sep 2025 08:27:39 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:
>> Hi all, >> >> The sub-test H3MalformedResponseTest.testMalformedResponse in >> test/jdk/java/net/httpclient/http3/H3MalformedResponseTest.java intermittent >> report java.util.concurrent.TimeoutException, espicial when this test run >> with other jtreg tests simultancely. This PR change the timeout vaule from >> 10 to 30 to make test run passed steady. > > test/jdk/java/net/httpclient/http3/H3MalformedResponseTest.java line 371: > >> 369: request, >> 370: BodyHandlers.discarding()) >> 371: .get(30, TimeUnit.SECONDS); > > @dfuch, @jaikiran, can we replace this with a simple `get()` and rely on > JTreg's default timeout settings? we probably could. This test should work equally well with a `send` instead of `sendAsync` too. But the proposed change is good enough for now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27447#discussion_r2372114756
