Skip to content

JDK8 Thread.cleanup requires TerminatingThreadLocal.threadTerminated() - #14069

Merged
pshipton merged 1 commit into
eclipse-openj9:masterfrom
JasonFengJ9:threadcleanup
Dec 3, 2021
Merged

JDK8 Thread.cleanup requires TerminatingThreadLocal.threadTerminated()#14069
pshipton merged 1 commit into
eclipse-openj9:masterfrom
JasonFengJ9:threadcleanup

Conversation

@JasonFengJ9

Copy link
Copy Markdown
Member

Replaced JAVA11 JPP decoration w/ Sidecar18-SE-OpenJ9 to allow JDK8 j.l.Thread.cleanup() to invoke TerminatingThreadLocal.threadTerminated() if required.
Sidecar18-SE (internal IBM build) is unchanged because they are still not updated to 1.8.0_322 yet.

Note: IBM Java 8 build is at jdk8u311-b11 which doesn't have jdk.internal.misc.TerminatingThreadLocal.

fixes #14008

Signed-off-by: Jason Feng fengj@ca.ibm.com

Replace JAVA11 JPP decoration w/ Sidecar18-SE-OpenJ9 to allow JDK8
j.l.Thread.cleanup() to invoke TerminatingThreadLocal.threadTerminated()
if required.
Sidecar18-SE (internal IBM build) is unchanged because they are still
not updated to 1.8.0_322.

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
@pshipton

pshipton commented Dec 3, 2021

Copy link
Copy Markdown
Member

jenkins compile alinux64 jdk8

@pshipton

pshipton commented Dec 3, 2021

Copy link
Copy Markdown
Member

I assume this is going to work, so pls cherry pick for the 0.30 branch.

@JasonFengJ9

Copy link
Copy Markdown
Member Author

Created #14070 for the v0.30.0-release branch.

@pshipton

pshipton commented Dec 3, 2021

Copy link
Copy Markdown
Member

Tested via grinder, which passed.
https://openj9-jenkins.osuosl.org/view/Test/job/Grinder/438

@pshipton
pshipton merged commit bd8ad84 into eclipse-openj9:master Dec 3, 2021
@JasonFengJ9
JasonFengJ9 deleted the threadcleanup branch April 4, 2022 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

2 participants