Skip to content

chore: upgrade minium Java version to 17#407

Merged
NguyenHoangSon96 merged 3 commits into
mainfrom
chore/upgrade-minium-jdk-version
Jul 1, 2026
Merged

chore: upgrade minium Java version to 17#407
NguyenHoangSon96 merged 3 commits into
mainfrom
chore/upgrade-minium-jdk-version

Conversation

@NguyenHoangSon96

@NguyenHoangSon96 NguyenHoangSon96 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Closes #

Proposed Changes

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • Tests pass
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@NguyenHoangSon96 NguyenHoangSon96 added this to the 1.8.0 milestone Jul 1, 2026
@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Jul 1, 2026
Copilot AI review requested due to automatic review settings July 1, 2026 07:29
@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.29%. Comparing base (0b9001f) to head (212ab7d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #407   +/-   ##
=======================================
  Coverage   88.29%   88.29%           
=======================================
  Files          21       21           
  Lines        1547     1547           
  Branches      280      280           
=======================================
  Hits         1366     1366           
  Misses         83       83           
  Partials       98       98           

☔ View full report in Codecov by Harness.
📢 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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR raises the project’s minimum supported Java version from 11 to 17 and updates build/CI/docs accordingly so compilation and the test matrix align with the new baseline.

Changes:

  • Update Maven compiler configuration to target Java 17.
  • Update the examples module to compile with Java 17.
  • Update CircleCI images and test matrix and adjust README minimum Java requirement.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
README.md Updates documented minimum Java requirement to 17.
pom.xml Bumps Maven compiler level to Java 17 for the main project build.
examples/pom.xml Bumps examples module compilation target to Java 17.
.circleci/config.yml Moves default CircleCI JDK image to 17 and drops JDK 11 from the test matrix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pom.xml Outdated
@NguyenHoangSon96 NguyenHoangSon96 force-pushed the chore/upgrade-minium-jdk-version branch from f1792ba to 6abb27c Compare July 1, 2026 07:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread CHANGELOG.md Outdated

@alespour alespour left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess move to JDK 17+ is inevitable, because next Java Apache Arrow v20 will also require 17+.

@NguyenHoangSon96 NguyenHoangSon96 merged commit ea0613b into main Jul 1, 2026
16 checks passed
@NguyenHoangSon96 NguyenHoangSon96 deleted the chore/upgrade-minium-jdk-version branch July 1, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants