Skip to content

Backporting Flink: Watermark Read Options to 1.17 and 1.16#9456

Merged
pvary merged 1 commit intoapache:mainfrom
rodmeneses:backPorting
Jan 13, 2024
Merged

Backporting Flink: Watermark Read Options to 1.17 and 1.16#9456
pvary merged 1 commit intoapache:mainfrom
rodmeneses:backPorting

Conversation

@rodmeneses
Copy link
Copy Markdown
Contributor

@rodmeneses rodmeneses commented Jan 10, 2024

Flink: Backports the Watermark Read Options to 1.17 and 1.16

The patch was cleanly applied to both 1.17 and 1.16
source:
git diff --no-index flink/v1.16/flink/src/main/java/org/apache/iceberg/flink/source flink/v1.18/flink/src/main/java/org/apache/iceberg/flink/source
brings some changes unrelated to this PR
git diff --no-index flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/source flink/v1.18/flink/src/main/java/org/apache/iceberg/flink/source
came out clean

test:
git diff --no-index flink/v1.16/flink/src/test/java/org/apache/iceberg/flink/source flink/v1.18/flink/src/test/java/org/apache/iceberg/flink/source
brings some changes on unrelated to this PR
git diff --no-index flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/source flink/v1.18/flink/src/test/java/org/apache/iceberg/flink/source
brings changes, some of them specifically related to junit Version

@github-actions github-actions Bot added the flink label Jan 10, 2024
@stevenzwu
Copy link
Copy Markdown
Contributor

@rodmeneses I assume you have checked the diff that nothing is related to this change, e.g. #8228

@pvary pvary merged commit 8109e42 into apache:main Jan 13, 2024
@pvary
Copy link
Copy Markdown
Contributor

pvary commented Jan 13, 2024

Thanks @rodmeneses for the backport, and @stevenzwu for the review!

geruh pushed a commit to geruh/iceberg that referenced this pull request Jan 26, 2024
adnanhemani pushed a commit to adnanhemani/iceberg that referenced this pull request Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants