Skip to content

Drop support for Flink 1.17, 1.18 and 1.19 - #7633

Closed
pan3793 wants to merge 2 commits into
apache:masterfrom
pan3793:kyuubi-drop-flink-1-17-1-18-1-19
Closed

Drop support for Flink 1.17, 1.18 and 1.19#7633
pan3793 wants to merge 2 commits into
apache:masterfrom
pan3793:kyuubi-drop-flink-1-17-1-18-1-19

Conversation

@pan3793

@pan3793 pan3793 commented Aug 14, 2026

Copy link
Copy Markdown
Member

Why are the changes needed?

Flink 1.17, 1.18 and 1.19 are EOL and were deprecated earlier (KYUUBI #7199, KYUUBI #7285),
with their building support already removed. Kyuubi 1.13 drops the remaining support, making
Flink 1.20 the minimum. Flink 2.0 is EOL, so it is marked deprecated.

What changes are proposed?

  • Remove the flink-1.17/flink-1.18/flink-1.19 Maven profiles and their deprecation warning
  • Drop Flink 1.17/1.18/1.19 from the CI matrix
  • Clean up stale adaptation code that only existed for Flink < 1.20, including the
    FlinkResultSet reflection shim, the 4-arg DefaultContext.load branch, and >= "1.17" test guards
  • Remove the obsolete kyuubi.engine.flink.doAs.generateTokenFile workaround, superseded by
    FLINK-35525 (1.20.0)
  • Deprecate Flink 2.0 at engine startup
  • Update docs to record the removal and deprecation

How was this patch tested?

Cross-version tested: engine compiled with Flink 1.20, run against a Flink 2.3.0 distribution;
FlinkOperationSuite and FlinkOperationSuiteOnYarn passed 12/12.

CI coverage: the flink-it matrix keeps Flink 1.20 (JDK 8) plus Flink 2.0/2.1/2.2/2.3
binary verification (JDK 17). Spotless and Scalastyle pass.

Was this patch authored or co-authored using generative AI tooling?

Assisted-by: DeepSeek V4 Pro

### Why are the changes needed?

Flink 1.17, 1.18 and 1.19 are EOL and were deprecated earlier (KYUUBI apache#7199, KYUUBI apache#7285),
with their building support already removed. Kyuubi 1.13 drops the remaining support, making
Flink 1.20 the minimum. Flink 2.0 is EOL, so it is marked deprecated.

### What changes are proposed?

- Remove the flink-1.17/flink-1.18/flink-1.19 Maven profiles and their deprecation warning
- Drop Flink 1.17/1.18/1.19 from the CI matrix
- Clean up stale adaptation code that only existed for Flink < 1.20, including the
  FlinkResultSet reflection shim, the 4-arg DefaultContext.load branch, and >= "1.17" test guards
- Deprecate Flink 2.0 at engine startup
- Update docs to record the removal and deprecation

### How was this patch tested?

Cross-version tested: engine compiled with Flink 1.20, run against a Flink 2.3.0 distribution;
FlinkOperationSuite and FlinkOperationSuiteOnYarn passed 12/12.

CI coverage: the flink-it matrix keeps Flink 1.20 (JDK 8) plus Flink 2.0/2.1/2.2/2.3
binary verification (JDK 17). Spotless and Scalastyle pass.

### Was this patch authored or co-authored using generative AI tooling?

Assisted-by: DeepSeek V4 Pro
@github-actions github-actions Bot added kind:documentation Documentation is a feature! kind:infra license, community building, project builds, asf infra related, etc. module:server module:flink kind:build labels Aug 14, 2026
@pan3793
pan3793 requested a review from wForget August 14, 2026 07:37

@wForget wForget left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @pan3793

@pan3793 pan3793 self-assigned this Aug 14, 2026
@pan3793 pan3793 added this to the v1.13.0 milestone Aug 14, 2026
FLINK-35525 (1.20.0) allows the kyuubi delegation token provider to pass tokens to the
YARN AM, and Flink 1.20 is now the minimum supported version, so the temporary
HADOOP_TOKEN_FILE workaround is no longer needed.
@pan3793

pan3793 commented Aug 17, 2026

Copy link
Copy Markdown
Member Author

thanks, merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:build kind:documentation Documentation is a feature! kind:infra license, community building, project builds, asf infra related, etc. module:common module:flink module:server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants