- Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-19668. Add SubjectInheritingThread and update Daemon to restore pre JDK22 Subject behaviour in Threads #8061
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
base: trunk
Are you sure you want to change the base?
Conversation
…e pre JDK22 Subject behaviour in Threads
pan3793 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.
LGTM, cc @steveloughran @slfan1989
| 💔 -1 overall
This message was automatically generated. |
I’ve shared my feedback in #7892. I’m giving this PR a soft +1, but I think we should remain patient for now. Thanks again for everyone’s efforts. |
| @stoty Could we also include the Javadoc updates and the new SpotBugs fixes? Thanks a lot! |
| I've checked the reported Javadoc errors, both are for files that I haven't touched. |
| I've also checked the two new SpotBugs warnings. |
Description of PR
First part of HADOOP-19574.
Add SubjectInheritingThread and update Daemon, but only update the classes using Daemon
How was this patch tested?
Tested together with HADOOP-19670.
Ran the full test suite with JDK25, and confirmed that it does not cause regressions
(i.e. only tests which also fail with Java 17 fail with Java 25)
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?