RFR: 8328366: Thread.setContextClassloader from thread in FJP commonPool task no longer works after JDK-8327501
Mandy Chung
mchung at openjdk.org
Thu Apr 4 16:21:10 UTC 2024
On Thu, 4 Apr 2024 14:46:11 GMT, Viktor Klang <vklang at openjdk.org> wrote:
>> src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 1143:
>>
>>> 1141: @SuppressWarnings("removal")
>>> 1142: SecurityManager sm = System.getSecurityManager();
>>> 1143: if (sm != null && isCommon)
>>
>> For common thread pool, if SM is not enabled, it will create `ForkJoinWorkerThread` that does not clear thread locals which is different than JDK 18 behavior. Is this intentional?
>
> @mlchung I think we should proceed with merging this as-is and look to address the ThreadLocal situation (or not at all, based on @DougLea's input). Are you OK with reviewing this as-is?
Yes it's fine to separate it as a follow up.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18374#discussion_r1552027346
More information about the core-libs-dev
mailing list