Skip to content

[FLINK-37659][release] Generate reference data for state migration tests based on release-2.0.0 #26442

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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

X-czh
Copy link
Contributor

@X-czh X-czh commented Apr 11, 2025

Steps:

  1. Update the content of apache/flink:flink-test-utils-parent/flink-migration-test-utils/src/main/resources/most_recently_published_version to v2_0.
  2. Run on release-2.0 and cherry pick changes back:
mvn clean package -Pgenerate-migration-test-data -Dgenerate.version=2.0 -nsu -Dfast -DskipTests

@flinkbot
Copy link
Collaborator

flinkbot commented Apr 11, 2025

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@davidradl
Copy link
Contributor

@X-czh I assume we have generated reference data for state migration tests based on 20. Do any of these work when going to v2? Or does the Kryo v2 ->v5 effect a large number of these tests?

@X-czh
Copy link
Contributor Author

X-czh commented Apr 15, 2025

@davidradl All except the Kryo ones work. The Kryo related tests have been intentionally disabled in #26243 (see the comments in KryoSerializerUpgradeTest for more details) and will be reenabled again in #26443 (only testing migration starting from v2.0).

@reswqa
Copy link
Member

reswqa commented Apr 15, 2025

+cc @Zakelly

Copy link
Contributor

@Zakelly Zakelly left a comment

Choose a reason for hiding this comment

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

@reswqa
Copy link
Member

reswqa commented Apr 17, 2025

Thanks for picking up this, merging...

@reswqa reswqa merged commit 7c03bc8 into apache:master Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants