Skip to content

Conversation

@turboFei
Copy link
Member

@turboFei turboFei commented Nov 5, 2025

Why are the changes needed?

Close #3531

This PR removes support for Java 8 across the project. Reasons:
Java 8 is end-of-life and no longer receives public updates or security patches.
Modern libraries and frameworks increasingly require Java 11+.
Dropping Java 8 simplifies code maintenance and allows use of newer language features and APIs.
Improves compatibility with current cloud and container environments, which default to Java 11 or higher.

Brief change log

  • Remove GA and deprecate Java8

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions bot added type:docs Improvements or additions to documentation type:infra type:build labels Nov 5, 2025
@codecov-commenter
Copy link

codecov-commenter commented Nov 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 28.94%. Comparing base (17b68d0) to head (a403e20).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3899      +/-   ##
============================================
+ Coverage     28.82%   28.94%   +0.11%     
- Complexity     3838     3847       +9     
============================================
  Files           625      631       +6     
  Lines         50514    50722     +208     
  Branches       6455     6496      +41     
============================================
+ Hits          14560    14680     +120     
- Misses        34928    34985      +57     
- Partials       1026     1057      +31     
Flag Coverage Δ
core 28.94% <ø> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xxubai
Copy link
Contributor

xxubai commented Nov 11, 2025

Should we drop support for Java8 in the next release? @zhoujinsong @baiyangtx @klion26

@turboFei
Copy link
Member Author

is there an ETA for next release?

@zhoujinsong
Copy link
Contributor

I am okay to drop JDK8 in the next version, if it keeps us from uprading the iceberg version.

@klion26
Copy link
Member

klion26 commented Nov 12, 2025

As we have not deprecated JDK8 in 0.8.x, maybe we need to deprecate the jdk8 in 0.9.x and remove it in the version after 0.9.x(0.10.x or 1.0.x ). If we want to remove the JDK8 support more quickly, maybe we can make an earlier release for this.

@turboFei
Copy link
Member Author

maybe we can make an earlier release for this.

+1. In my view, the legacy burden of JDK 8 has become a hindrance to the advancement of the community.

@github-actions
Copy link

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 13, 2025
@czy006
Copy link
Contributor

czy006 commented Dec 13, 2025

As we have not deprecated JDK8 in 0.8.x, maybe we need to deprecate the jdk8 in 0.9.x and remove it in the version after 0.9.x(0.10.x or 1.0.x ). If we want to remove the JDK8 support more quickly, maybe we can make an earlier release for this.

The removal of support for JDK8 has been approved through voting on the mailing list. I am in favor of this decision. Moving forward, with support for V3 or higher versions, we can no longer maintain compatibility with JDK8. It's time to bid farewell to it.

@turboFei turboFei closed this Dec 13, 2025
@turboFei turboFei reopened this Dec 13, 2025
@github-actions github-actions bot removed the stale label Dec 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:build type:docs Improvements or additions to documentation type:infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement]: Deprecate Java 8

6 participants