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

[HUDI-6508] Support compilation on Java 11 #11513

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

yihua
Copy link
Contributor

@yihua yihua commented Jun 25, 2024

Change Logs

This PR contains the changes from #11479 and targets at branch-0.x.

This PR adds support for compiling Hudi on Java 11 for engines like Spark and Flink that support Java 11 and 17. Detailed changes:

  • Fixes compilation errors on Java 11; maintains the code to be compilable on Java 8.
  • Adds new GitHub CI jobs on validating Hudi built by Java 11, by running the tests on Spark 3.5 with Java 17 runtime. These are moved from existing CI jobs validating Hudi built by Java 8. The tests on Spark 3.3 and 3.4 still run with Java 8 compilation and Java 17 runtime.

Kudos to @Zouxxyy for contributing to this PR from #9300.

Impact

Makes Hudi compilable on Java 11.

Risk level

none

Documentation Update

Docs on Java 11 compilation support.

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label Jun 25, 2024
@hudi-bot
Copy link

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@nsivabalan nsivabalan merged commit 538e661 into apache:branch-0.x Jun 26, 2024
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M PR with lines of changes in (100, 300]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants