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

[SUPPORT] Current state of Java 17 support #12207

Open
mzheng-plaid opened this issue Nov 5, 2024 · 3 comments
Open

[SUPPORT] Current state of Java 17 support #12207

mzheng-plaid opened this issue Nov 5, 2024 · 3 comments
Assignees
Labels

Comments

@mzheng-plaid
Copy link

I saw https://issues.apache.org/jira/browse/HUDI-6506 ("Java 11 and 17 support with Hudi") and it seems like this is only targeted towards 1.x release, though it seems like some work has been completed already for certifying runtime/compile support for Hudi on Java 17:

#9136 - it seems like CI has been running against Java 17 since 0.14.0
#11604 - "Make Hudi compilable with Java 17"

  1. What is the current state of Java 17 support for 0.14.x, 0.15.x, etc? We've done some tests on EMR and there's been no issues (and significant perf improvements in some test workloads), but wanted to check if there are any known issues.
  2. Has the EMR team backported Java 17 support to their forks of Hudi? On EMR 7.x the default JVM became Java 17 - is Hudi certified to work with Java 17 on EMR?
@mzheng-plaid
Copy link
Author

@CTTY I saw you worked on supporting Java 17 in those tickets - is this something you could help answer? Appreciate the work on this

@CTTY
Copy link
Contributor

CTTY commented Nov 12, 2024

Hi @mzheng-plaid, we've certified Java 17 runtime support since 0.14 and we haven't noticed any major issues so far. Java 17 compilation time has been done as well: #11604 . But the default Java version stays to be Java 8.

On EMR, Hudi 0.14/0.15 run on Java 17 by default.

Hopefully this can help answer the ask. Please feel free to create a ticket to EMR Hudi team if you have more questions regarding Hudi on EMR

@mzheng-plaid
Copy link
Author

Thanks appreciate the answer @CTTY that's very helpful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Awaiting Triage
Development

No branches or pull requests

4 participants