> 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