Skip to content

chore(build): Require Java 17+ for production code - #211

Closed
adutra wants to merge 1 commit into
dremio:mainfrom
adutra:java17
Closed

chore(build): Require Java 17+ for production code#211
adutra wants to merge 1 commit into
dremio:mainfrom
adutra:java17

Conversation

@adutra

@adutra adutra commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

// Task to verify that the production JAR is compatible with Java 11
val checkJava11Compatibility by
// Task to verify that the production JAR is compatible with Java 17
val checkJava17Compatibility by

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Are we compelled to bump the java version?

If not, it might be easier for clients to keep 11 🤔 WDYT?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We are not compelled, but since Iceberg itself is switching to a Java 17 baseline, I figured it was time to change this. But, OTOH, we can wait until Iceberg 1.11 is out. Let's put this one on hold.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ah... I see 🙂

@adutra
adutra marked this pull request as draft April 21, 2026 08:06
@adutra

adutra commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #240.

@adutra adutra closed this May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants