RFR: 8297200: java/net/httpclient/SpecialHeadersTest.java failed once in AssertionError due to selector thread remaining alive [v2]

Daniel Fuchs dfuchs at openjdk.org
Tue Nov 22 21:34:38 UTC 2022


> The java/net/httpclient/SpecialHeadersTest.java test has been observed failing once, due to a selector thread still running after all operations had terminated. This change slightly alter the test logic to check the shared client only at the end of the test when all methods have been executed, and focussed the checks performed by a test method to the only clients that this method is using. This may or may not prevent the observed issue to ever reproducing, but should at least provide more information on where the selector manager thread is at the time it is observed running, should the test fail again:
> the logic of the HttpClient should have woken up the selector when the last operation finished, which should have ensured a prompt termination of the thread.

Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:

  Fixed race condition in detecting if selector is still alive

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11294/files
  - new: https://git.openjdk.org/jdk/pull/11294/files/204f279c..25efbd8b

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

  Stats: 72 lines in 4 files changed: 59 ins; 0 del; 13 mod
  Patch: https://git.openjdk.org/jdk/pull/11294.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11294/head:pull/11294

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


More information about the net-dev mailing list