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
[STORM-3895] Add JDK-17 build #3555
Conversation
Very good initiative, thanks ! |
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.
looks like some builds are failing
…er - to allow reflection on java.base
@@ -169,6 +169,8 @@ | |||
<artifactId>maven-surefire-plugin</artifactId> | |||
<configuration> | |||
<forkCount>1</forkCount> | |||
<!-- remove this when ServerUtilsTest.getPidOfPosixProcess() returns p.pid() as allowed in JDK11 and above --> |
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'm not sure what this issue is, but I would file a JIRA for it and mention it in this comment so we can track progress.
… test under JDK17
For some reason, under openjdk-17, the license file is much smaller (and different) from that generated under jdk8 and jdk11. Wondering is this is related to strict isolation of packages and restrictions on reflection introduced in JDK17. |
… specific overrides based on JDK.
What is the purpose of the change
Add JDK-17 Build
How was the change tested
Let the github actions run