RFR: 8284199: Implementation of Structured Concurrency (Incubator) [v2]
Alan Bateman
alanb at openjdk.java.net
Tue May 24 11:47:03 UTC 2022
On Tue, 24 May 2022 10:52:07 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Alan Bateman 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 six additional commits since the last revision:
>>
>> - Use {@code ...}, replace task->subtask, fix typos, add jls ref
>> - Merge
>> - @ignore StructuredThreadDumpTest until test infra in place
>> - Refresh
>> - Merge
>> - Initial commit
>
> src/jdk.incubator.concurrent/share/classes/jdk/incubator/concurrent/StructuredTaskScope.java line 214:
>
>> 212: * <h2><a id="TreeStructure">Tree structure</a></h2>
>> 213: *
>> 214: * StructuredTaskScopes form a tree where parent-child relations are established
>
> Should we mention what happens in the owner thread completes its execution and the scope's `close` method has not been called? I think that, as discussed offline, the fact that the thread will attempt to close any nested scopes when terminating is an important aspect of this API.
The close method might be the right place for this. It has to specify that an attempt to close out of order will close the nested scopes and terminating without close is much the same. I'll see what I can do, thanks for this suggestion.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8787
More information about the core-libs-dev
mailing list