On Mon, 14 Aug 2023 11:52:00 GMT, Doug Lea <d...@openjdk.org> wrote:
>> 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 45 additional commits since
> the last revision:
>
> - Merge branch 'openjdk:master' into JDK-8288899
> - Update @since tags
> - resync
> - Merge branch 'openjdk:master' into JDK-8288899
> - ExecutorService tests
> - Merge branch 'openjdk:master' into JDK-8288899
> - Rework versioning
> - Merge branch 'openjdk:master' into JDK-8288899
> - Merge branch 'openjdk:master' into JDK-8288899
> - Merge branch 'openjdk:master' into JDK-8288899
> - ... and 35 more: https://git.openjdk.org/jdk/compare/a26bcff0...585769c9
src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 721:
> 719: * they are ignored, and 0 is an OK default.) The scan in method
> 720: * helpJoin uses these markers to try to find a worker to help
> 721: * (i.e., steal back a task from and execute it) that could makje
Suggestion:
* (i.e., steal back a task from and execute it) that could make
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14301#discussion_r1296996791
