Skip to content
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

JDK 11 CI Builds #3301

Closed
pomadchin opened this issue Nov 4, 2020 · 4 comments · Fixed by #3435
Closed

JDK 11 CI Builds #3301

pomadchin opened this issue Nov 4, 2020 · 4 comments · Fixed by #3435

Comments

@pomadchin
Copy link
Member

pomadchin commented Nov 4, 2020

Java 11 is the current LTS and accroding to https://docs.scala-lang.org/overviews/jdk-compatibility/overview.html is very well supported by Scala.

Even though all major builds still should remain built against Java 8, we need to consider adding the Java 11 support builds, at least to remain up to date. The current master can't be compiled with Java 11+.

Requires upgrade up to Hadoop 3.3 and all deps that depend on lower hadoop versions, see https://cwiki.apache.org/confluence/display/HADOOP/Hadoop+Java+Versions

@pomadchin
Copy link
Member Author

This issue turned out to be a bit more complicated, since not only the codebase changes are required but also some libs deps updates.

@pomadchin pomadchin changed the title Java 11 builds support JDK 11 CI Builds Apr 13, 2021
@pomadchin
Copy link
Member Author

We support JDK11, see #3383

Let's reframe this issue into adding the CI that would run tests with JDK 11.

@pomadchin
Copy link
Member Author

pomadchin commented Jul 14, 2022

I think we can start seriously thinking about dropping JDK8 builds, EMR supports JDK 11 already (https://docs.aws.amazon.com/emr/latest/ReleaseGuide/configuring-java8.html (guide to configure JDKs) + https://stackoverflow.com/questions/70886684/how-to-use-java-runtime-11-in-emr-cluster-aws)

cc @echeipesh @metasim this descision affects RasterFrames future releases

@metasim
Copy link
Member

metasim commented Jul 21, 2022

@pomadchin Thanks for the head's up. It's been a long time coming, and definitely needed. Not sure when I'll get to an RF build tho. :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants