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

Wed Aug 3 14:37:47 UTC 2022

On Wed, 3 Aug 2022 14:33:56 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> 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

Marked as reviewed by jpai (Reviewer).

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

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