RFR: 8285836: sun/net/www/http/KeepAliveCache/KeepAliveProperty.java failed with "RuntimeException: Failed in server" [v2]

Daniel Jeliński djelinski at openjdk.org
Wed Aug 3 14:33:56 UTC 2022


> This patch fixes a race condition in KeepAliveProperty test. The client thread could read the `pass` field and fail the test before the server thread had a chance to set the field value to `true`. The fix adds an explicit wait for the server thread to complete.
> 
> Without this patch, the failure can be easily reproduced by adding a `Thread.sleep(100);` before `pass = !expectClose;`.

Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:

  Make logger static

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9700/files
  - new: https://git.openjdk.org/jdk/pull/9700/files/e2754b47..10ca712d

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

  Stats: 2 lines in 1 file changed: 1 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/9700.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9700/head:pull/9700

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


More information about the net-dev mailing list