RFR: 8362123: ClassLoader Leak via Executors.newSingleThreadExecutor(...) [v2]

[Chris Dennis]

On Fri, 26 Sep 2025 16:42:49 GMT, Chris Dennis <duke at openjdk.org> wrote:

>> Executors shutdown via `shutdownNow()` should have their cleanables cleaned to prevent a classloader leak. This can happen if a classloader exists that both references the wrapped executor and is referenced by the delegate executor.
>> 
>> To quote @Martin-Buchholz:
>>> BTW: I find Cleaners much harder to use than old finalize, and it looks like I'm not the only one!
>
> Chris Dennis has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
> 
>   8362123: ClassLoader Leak via Executors.newSingleThreadExecutor(...)
>   
>   Executors shutdown via `shutdownNow()` should have their cleanables cleaned to
>   prevent a classloader leak. This can happen if a classloader exists that both
>   references the wrapped executor and is referenced by the delegate executor.

Apologies gentlemen, but do I need to do something to move this on? If I understand correctly it's just a `/integrate` command to the Skara machinery now, but I don't believe I have the permissions to trigger that?

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

PR Comment: https://git.openjdk.org/jdk/pull/26296#issuecomment-3382096547