Skip to content

[FLINK-38729] Bump Flink version to 2.2.0#4287

Closed
lvyanquan wants to merge 1 commit into
apache:masterfrom
lvyanquan:FLINK-38729-2
Closed

[FLINK-38729] Bump Flink version to 2.2.0#4287
lvyanquan wants to merge 1 commit into
apache:masterfrom
lvyanquan:FLINK-38729-2

Conversation

@lvyanquan
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 adds support for Apache Flink 2.2.0 alongside the existing Flink 1.20.3 support by introducing a new module flink-cdc-common-2.x. The changes enable the Flink CDC project to support both Flink 1.x and 2.x versions concurrently.

Changes:

  • Adds version properties for both Flink 1.x (1.20.3) and Flink 2.x (2.2.0) in pom.xml
  • Updates shaded Guava versions for both Flink versions (31.1-jre-17.0 for 1.x, 33.4.0-jre-20.0 for 2.x)
  • Changes annotation imports from org.apache.flink.annotation to org.apache.flink.cdc.common.annotation in existing files
  • Creates new flink-cdc-common-2.x module with Flink 2.x specific implementations

Reviewed changes

Copilot reviewed 38 out of 38 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pom.xml Adds version properties for dual Flink version support and new module
flink-cdc-common/src/main/java/org/apache/flink/cdc/common/utils/ThreadLocalCache.java Updates @Internal annotation import to CDC's annotation package
flink-cdc-common/src/main/java/org/apache/flink/cdc/common/types/VariantType.java Updates @PublicEvolving annotation import to CDC's annotation package
flink-cdc-common-2.x/pom.xml New module configuration for Flink 2.x support
flink-cdc-common-2.x/src/main/java/* New implementations using Flink 2.x APIs (variant types, binary formats, factories, etc.)
flink-cdc-common-2.x/src/test/java/* Comprehensive test coverage for new implementations

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

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.

2 participants