> 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

Reply via email to