Integrated: 8285836: sun/net/www/http/KeepAliveCache/KeepAliveProperty.java failed with "RuntimeException: Failed in server"

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


On Mon, 1 Aug 2022 11:01:01 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;`.

This pull request has now been integrated.

Changeset: af529be0
Author:    Daniel Jeliński <djelinski at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/af529be069b651808310c3c37e2167d216979f69
Stats:     3 lines in 1 file changed: 2 ins; 1 del; 0 mod

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

Reviewed-by: dfuchs, jpai

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

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


More information about the net-dev mailing list