> Nothing looks wrong with the test or the code. The failure happens rarely - 
> probably when the machine is under load: this test tries to saturate the 
> socket buffers and is resource consuming.
> 
> The proposed fix is just to double the jtreg timeout for this test from 120 
> to 240.

Daniel Fuchs 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 six additional commits since the 
last revision:

 - Review feedback: the issue is the logic in byteArrayOfSize
 - Merge branch 'master' into RawChannel-8371366
 - Fix copyright dates
 - Review feedback: increasing the timeout might not help
 - Merge branch 'master' into RawChannel-8371366
 - 8371366: java/net/httpclient/whitebox/RawChannelTestDriver.java fails 
intermittently in jtreg timeout

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28178/files
  - new: https://git.openjdk.org/jdk/pull/28178/files/e05864db..18203dab

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28178&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28178&range=02-03

  Stats: 13718 lines in 142 files changed: 9233 ins; 3636 del; 849 mod
  Patch: https://git.openjdk.org/jdk/pull/28178.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28178/head:pull/28178

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

Reply via email to