-
Notifications
You must be signed in to change notification settings - Fork 86
Update getState() API to use holder.threadStatus field #927
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
90b92ae to
59625ad
Compare
|
@keithc-ca is this waiting for your review? |
Yes, but more VM changes are coming, see eclipse-openj9/openj9#21043 (comment). |
59625ad to
e81c84d
Compare
keithc-ca
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase to avoid implicit merges.
cae2639 to
a5c22c6
Compare
Co-authored-by: Nathan Henderson <nathan.henderson@ibm.com> Signed-off-by: Jack Lu <Jack.S.Lu@ibm.com>
a5c22c6 to
2273950
Compare
|
Jenkins test sanity zlinux jdknext depends eclipse-openj9/openj9#21043 |
keithc-ca
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the test failures are related to this change.
depend: eclipse-openj9/openj9#21043
Rework of #879