RFR: 8288899: java/util/concurrent/ExecutorService/CloseTest.java failed with "InterruptedException: sleep interrupted" [v32]

Doug Lea dl at openjdk.org
Wed Sep 13 16:19:36 UTC 2023


> Addresses Jdk 8288899 : java/util/concurrent/ExecutorService/CloseTest.java failed with "InterruptedException: sleep interrupted" and related issues.
> 
> This is a major ForkJoin update (and hard to review -- sorry) that finally addresses incompatibilities between ExecutorService and ForkJoinPool (which claims to implement it), with the goal of avoiding continuing bug reports and incompatibilities. Doing this required reworking internal control to use phaser/seqlock-style versioning schemes (affecting nearly every method) that ensure consistent data structures and actions without requiring global synchronization or locking on every task execution that would massively degrade performance. The previous lack of a solution to this was the main reason for these incompatibilities.

Doug Lea has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 77 additional commits since the last revision:

 - Merge branch 'openjdk:master' into JDK-8288899
 - Strengthen translation of getAndX atomics; revert or adapt FJP accordingly
 - Merge branch 'openjdk:master' into JDK-8288899
 - Force more orderings; improve diagnostics
 - Simplify signalling
 - Always help terminate when stopping
 - Merge branch 'openjdk:master' into JDK-8288899
 - Use non-recursive tasks in close tests
 - Allow ThreadGroup access in tck tests
 - Avoid needing test threads
 - ... and 67 more: https://git.openjdk.org/jdk/compare/35251845...d04ce80f

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/14301/files
  - new: https://git.openjdk.org/jdk/pull/14301/files/e8d7b75f..d04ce80f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14301&range=31
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14301&range=30-31

  Stats: 5410 lines in 768 files changed: 2649 ins; 1894 del; 867 mod
  Patch: https://git.openjdk.org/jdk/pull/14301.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14301/head:pull/14301

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


More information about the core-libs-dev mailing list