Drop Java 8 support and move to Java 11/17? #13835
Replies: 14 comments 17 replies
-
|
Spark versions that can work with Java 17.
Presto/Trino have moved onto Java 21+ or so.. So its just Flink. that will drag us back to Java 11. What flink version is most prevalently used with Hudi? @danny0405 ? |
Beta Was this translation helpful? Give feedback.
-
|
From @yihua on the dev sync. |
Beta Was this translation helpful? Give feedback.
-
|
Hive and Java Compatibility:
|
Beta Was this translation helpful? Give feedback.
-
|
Had a sync with @cshuo and @danny0405 today and we have came up with the following conclusion: If Java 17 and Java 11 is compatible, we can upgrade to Java 17 directly. However, most Chinese users JDK 8, we might need to provide users with a guide on how to perform manual compilation JDK8. |
Beta Was this translation helpful? Give feedback.
-
|
+1 to move to java 17, and providing a java 8 compilation guide. |
Beta Was this translation helpful? Give feedback.
-
|
@xushiyan @voonhous Please clarify my question above. What's the benefit of the move? |
Beta Was this translation helpful? Give feedback.
-
|
My question still is : what do we gain by making java 17 as default if we stick to java 8 language level. How'd we take advantage of the latest language features. I'd love to drop support for Java 8 and move to Java 11 or 17. |
Beta Was this translation helpful? Give feedback.
-
|
@danny0405 @voonhous Are we saying we are keeping code language level at 8? |
Beta Was this translation helpful? Give feedback.
-
|
To summarize: we should either drop support for Java 8 or stay at Java 8 as default. Looks like we are saying there are still Flink users on java 8, that we'd like to support |
Beta Was this translation helpful? Give feedback.
-
|
@voonhous @cshuo @danny0405 - can we circulate this discussion on Flink forums and see if users are ok with dropping java 8? I'd like to def drop java 8 and move language level to Java 11 atleast in 2.0 |
Beta Was this translation helpful? Give feedback.
-
|
Unless we hear strong appeals, I propose to end java 8 support with the 1.x release line. There are lots of latest features in jdk, for the RFC-100 type work. |
Beta Was this translation helpful? Give feedback.
-
|
+1 on ending JDK 8 support from the next major 1.x release. Given recent findings that RFC-100 (unstructured data support in Hudi with file format including Lance) requires JDK 11 (#14128), keeping Java 8 support blocks new features and future development. Also, users requiring Java 8 can still upgrade to Hudi 1.1. I propose that we move to JDK 11 as the minimum for Hudi 1.2 release as a first step, and then move to JDK 17 (which requires Scala version upgrade and other major changes based on my previous experience) as the minimum in another release. |
Beta Was this translation helpful? Give feedback.
-
|
We have voted on the DEV list to drop java 8, in 1.2. |
Beta Was this translation helpful? Give feedback.
-
|
To clarify the support matrix, in the next release Hudi 1.2, since we're dropping java 8, moving to 11 or 17 has different implications of engine integration compatibility. Below is a support matrix:
Based on flink 1.18-1.20 only having beta support to work with java 17, moving to java 11 is safer. Also note that Hive support needs further verification, going with java 11 or 17 would require extra setup or dropping support before hive 4.1. I propose moving to java 11 as default language level and compile target in Hudi 1.2, this means dropping flink 1.17, hive 3.x & 4.0.x support (if no hadoop upgrade workaround). |
Beta Was this translation helpful? Give feedback.

Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Could we skip 11 and go directly to 17? Flink only supports Java 17 from 1.18 onwards?
Thoughts? Considerations? Is anyone still on Java 8 - or have objections to dropping java 8 ..
Here's some WIP PR on moving to 11. any takers to get this to finish line (assuming we are okay doing so as a group)
#12880
Beta Was this translation helpful? Give feedback.
All reactions