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

[FLINK-35584][docker] Support Java21 in v1.20.0 #203

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JEnglandOhalo
Copy link

Add docker files for that Java 21 can be used with version 1.20.0 of Apache Flink.

Add docker files for that Java 21 can be used with version 1.20.0 of
Apache Flink.
@priyankasunilnairtrimble

@reswqa I noticed that most of the recent commits were done by you. Are there any plans to support this change any time soon so that we have a docker image for Flink 1.20 that supports Java 21?

@reswqa
Copy link
Member

reswqa commented Jan 22, 2025

I didn't learn about any plans for 1.20 to support jdk21, or maybe I missed some discussion.

cc @snuyanzin as you've done most of the work for jdk21 support.

@JEnglandOhalo
Copy link
Author

I didn't learn about any plans for 1.20 to support jdk21, or maybe I missed some discussion.

cc @snuyanzin as you've done most of the work for jdk21 support.

I understand that Flink 1.19 added beta support for Java 21 - see https://www.confluent.io/en-gb/blog/exploring-apache-flink-1-19/

But, unfortunately, no OCI containers were released which meant we couldn't integrate with it in our codebase and so had to stay on Java 17.

@snuyanzin
Copy link

snuyanzin commented Jan 22, 2025

we had all the tests cron jobs for jdk21 since 1.19
the last bit (for development) is enablement of spotless done in 2.0 since this task required dropping of jdk 1.8

At the same time there is still an activity FLINK-33699 which finally was decided not to do however still. makes sense to keep in mind.
There is no tests for savepoints stored with jdk 1.8/11/17 and restored in jdk21

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.

4 participants