On Tue, 29 Oct 2024 06:32:59 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> src/java.base/share/classes/java/util/concurrent/Executors.java line 529:
>>
>>> 527: * execute the given {@code callable} under the current access
>>> 528: * control context, with the current context class loader as the
>>> 529: * context class loader. This method should normally be invoked
>>
>> We no longer state that the context class loader will be set when the
>> callable is called, though the returned Callable _will_ still set the ccl.
>> Is this OK?
>
> That's a good observation. Although deprecated for removal, the wording
> should at least specify what it does for the period before it is removed.
> I'll come up with some spec wording for this.
>
> Update: now fixed in sandbox
Fixed in
https://github.com/openjdk/jdk/pull/21498/commits/0feceaae472710aa13e4c569f4d72d6149b7de07
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1820738332
