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

Flink: Backporting PR 7161 of custom partitioner for bucket partitions #8328

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

kengtin
Copy link
Contributor

@kengtin kengtin commented Aug 15, 2023

Backporting PR #7161 (Flink: Custom partitioner for bucket partitions) from 1.16 -> 1.15/1.17

The original PR was a 9-file change, ported to 1.15 and 1.17 so 18 affected files total on this one.

Verified contents (no difference/changes):

git diff --no-index  flink/v1.16/flink/src/main/java/org/apache/iceberg/flink/ flink/v1.15/flink/src/main/java/org/apache/iceberg/flink/
git diff --no-index  flink/v1.16/flink/src/main/java/org/apache/iceberg/flink/ flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/```

@github-actions github-actions bot added the flink label Aug 15, 2023
@kengtin kengtin changed the title Backporting PR 7161 from 1.16 -> 1.15/1.17 Backporting PR 7161 (Flink: Custom partitioner for bucket partitions) from 1.16 -> 1.15/1.17 Aug 15, 2023
@stevenzwu
Copy link
Contributor

@kengtin just to confirm that you have done git diff and see no diffs related to this change after back port, right?

@kengtin
Copy link
Contributor Author

kengtin commented Aug 17, 2023

Yes @stevenzwu I verified no differences via (added to the description too):

git diff --no-index  flink/v1.16/flink/src/main/java/org/apache/iceberg/flink/ flink/v1.15/flink/src/main/java/org/apache/iceberg/flink/
git diff --no-index  flink/v1.16/flink/src/main/java/org/apache/iceberg/flink/ flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/```

@stevenzwu stevenzwu changed the title Backporting PR 7161 (Flink: Custom partitioner for bucket partitions) from 1.16 -> 1.15/1.17 Flink: Backporting PR 7161 of custom partitioner for bucket partitions Aug 18, 2023
@stevenzwu stevenzwu merged commit 21acd34 into apache:master Aug 18, 2023
13 checks passed
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.

None yet

2 participants