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

[common] Bump flink to 1.17.0 #2057

Merged
merged 6 commits into from
May 26, 2023
Merged

[common] Bump flink to 1.17.0 #2057

merged 6 commits into from
May 26, 2023

Conversation

snuyanzin
Copy link
Contributor

The PR to move to 1.17.0 flink

@ruanhang1993
Copy link
Contributor

Hi, @snuyanzin . Thanks for this PR.
Could you refer to the PR #1696 to update the documents and e2e tests? Thanks ~

@ruanhang1993
Copy link
Contributor

Hi, @snuyanzin.
There are some failed tests. Would you mind to take a look at it ?
Thanks~

@ruanhang1993 ruanhang1993 changed the title Bump flink to 1.17.0 [common] Bump flink to 1.17.0 May 25, 2023
@ruanhang1993 ruanhang1993 self-requested a review May 25, 2023 09:06
@snuyanzin
Copy link
Contributor Author

sorry for the delay
looks like i missed it
thanks for fixing tests!

@ruanhang1993
Copy link
Contributor

sorry for the delay looks like i missed it thanks for fixing tests!

Never mind. This PR could be merged after this CI passes.

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

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

LGTM.

@ruanhang1993 ruanhang1993 merged commit c3a88f0 into apache:master May 26, 2023
10 of 13 checks passed
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.

None yet

2 participants