-
Notifications
You must be signed in to change notification settings - Fork 500
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
#4259 support Java 11 #7514
#4259 support Java 11 #7514
Conversation
…ava_11 remove java 8 profile
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.
pom.xml
Outdated
@@ -43,7 +43,7 @@ | |||
Jacoco 0.8.2 seems to break Netbeans code coverage integration so we'll use 0.8.1 instead. |
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.
this comment is no longer needed, no?
@djbrooke - the only things I think needs to be in the notes are
I haven't checked but I assume there's a prerequisite entry for Java that should be updated as well? (W.r.t. guides, since starting from Payara's default domain will get you the jvm-options already, I don't think any mention of those are needed in the guide, just in upgrade instructions, so the guide change should be simple). I can try to provide final text, but hopefully you can take the above and create release notes> I'll check to see where the guide needs updates at least. |
@qqmyers thanks for the details, I'll give it a shot! |
Awesome - release notes sound so natural when you write them! |
@qqmyers thanks! @scolapasta if the release notes look OK to you, can you approve so we can move to QA? |
…ava_11 updating Java 1.8 mentions to Java 11
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.
Should we also update Vagrant and docker-aio to Java 11?
…eeds more updating in its own issue
…erse into 4259_java_11
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 great. Java 11, here we come!
What this PR does / why we need it: This PR makes the changes necessary to update the Dataverse application and tests to run under Java 11. That's generally a good idea, allows us to take advantage of newer Java features, and is currently a prerequisite for #7504 and #7414 (these include a library built for Java 11). Note that running payara on Java 11 requires some jvm option changes that are in the default domain but would not be in domains upgraded per our instructions from glassfish. The changes will need to be documented for the release notes.
Which issue(s) this PR closes:
Closes #4259
Special notes for your reviewer: At the time of the original issue, it was less clear which Java version to upgrade to, but today, it looks like Java 11 (open) is widely available (e.g. default on Ubuntu 18). The primary issues seen were:
Suggestions on how to test this: Should be able to run Dataverse, unit and integration tests without problems on Java 11. One way to do that is with dataverse-ansible, setting the Java version to 11 and maven version to 3.6.3.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes update needed for this change?:
The following changes are needed to run with an upgraded domain.xml. They can be made using the asadmin interface or by editing the domain.xml file. Notes should include whichever method is decided on:
remove (J8 -specific):
Add (Java 9+ specific):
Additional documentation: