RFR: 8374432: TimeoutResponseBodyTest.java#retriesEnabledForResponseFailure fails run with -Xcomp [v2]

SendaoYan syan at openjdk.org
Sat Jan 3 02:08:21 UTC 2026


> Hi all,
> 
> The test test/jdk/java/net/httpclient/TimeoutResponseBodyTest.java#retriesEnabledForResponseFailure  was observed timed out when run with jvm option -Xcomp. The failure details shows in [JDK-8374432](https://bugs.openjdk.org/browse/JDK-8374432). Maybe it need more time for C2 to compile the httpclient classes than the default mixed mode.
> 
> So this PR read the timeout factor from jtreg, and apply the timeout factor to `REQUEST_TIMEOUT` by multiply it. Change has been verified locally by run the related tests test/jdk/java/net/httpclient/TimeoutResponseHeaderTest.java and test/jdk/java/net/httpclient/TimeoutResponseBodyTest.java both with -Xmixed and -Xcomp. Test-fix only, no risk,

SendaoYan has updated the pull request incrementally with one additional commit since the last revision:

  use "jdk.test.lib.Utils.adjustTimeout"

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/29012/files
  - new: https://git.openjdk.org/jdk/pull/29012/files/12b21e72..9c7e744e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=29012&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29012&range=00-01

  Stats: 8 lines in 1 file changed: 1 ins; 3 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/29012.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29012/head:pull/29012

PR: https://git.openjdk.org/jdk/pull/29012


More information about the net-dev mailing list