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

End-of-life Flink 1.16 is still referenced in docs #10412

Closed
manuzhang opened this issue Jun 1, 2024 · 3 comments · Fixed by #10463
Closed

End-of-life Flink 1.16 is still referenced in docs #10412

manuzhang opened this issue Jun 1, 2024 · 3 comments · Fixed by #10463
Labels
bug Something isn't working

Comments

@manuzhang
Copy link
Contributor

Apache Iceberg version

main (development)

Query engine

Flink

Please describe the bug 🐞

Flink 1.16 is no longer supported, but it's still referenced in the nightly docs. For example, at https://iceberg.apache.org/docs/nightly/flink/#preparation-when-using-flink-sql-client

Download Flink from the Apache download page. Iceberg uses Scala 2.12 when compiling the Apache iceberg-flink-runtime jar, so it's recommended to use Flink 1.16 bundled with Scala 2.12.

There is another link to https://repo.maven.apache.org/maven2/org/apache/iceberg/iceberg-flink-runtime-1.16/1.5.2/ on this page, and it won't work after next feature release (1.6.0).

We can easily forget to update these docs when Flink versions reaching end-of-life. However, I haven't figured out a better alternative. Thoughts? @nastra @Fokko @pvary @stevenzwu

@manuzhang manuzhang added the bug Something isn't working label Jun 1, 2024
@ajantha-bhat
Copy link
Member

While removing the module/folder during EOL marking, we should look up the documents and update it aswell (Reviewer also should check this).

We (I) handled it for Spark 3.1 and 3.2 while marking EOL. Maybe we missed for latest Flink update.

@Fokko
Copy link
Contributor

Fokko commented Jun 3, 2024

@manuzhang Thanks for bringing this up 🙌

What about making the version configurable, both for Spark and Flink, similar to what we did for Iceberg and Nessie.

@manuzhang
Copy link
Contributor Author

Yes, I'm thinking about introducing a DEFAULT_FLINK_VERSION as we do for the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants