CASSJAVA-135 Update the build to use Java17 rather than Java8 - #2098
CASSJAVA-135 Update the build to use Java17 rather than Java8#2098absurdfarce wants to merge 2 commits into
Conversation
patch by Bret McGuire; reviewed by X and Y
|
Jenkins run on this new branch was also green (minus the PointIT issues which should be addressed by #2097) |
|
Ping @tolbertam for visibility after moving to a new PR |
SiyaoIsHiding
left a comment
There was a problem hiding this comment.
This will break ASF CI, right?
|
Hmmm, we might need to make some changes to the ASF CI to get it to build with Java17 as a default. But we probably need a general cleanup there anyway since I don't think the Java25 changes included any updates to the ASF Jenkinsfile either. Might be worth just having a general "update the ASF CI" PR once this goes in. |
|
What do I need to do locally to run |
|
Are you sure you switched to Java17 @SiyaoIsHiding? We switch to using the maven.compiler.release property in this PR since that's the standard way of notating this for "newer" JDKs (by which they apparently mean anything older than Java9). The error you're seeing seems to suggest that your javac version can't support "-release 17" which in turn makes me think you might still be using an older JDK. To validate I executed |
|
You are right! Turns out because I have both |
Replaces #2096