-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[hotfix] Change old com.ververica dependency to flink #3110
Conversation
Hi @xleoken, Can you update flink-cdc.sh too? |
@xleoken Hi,I‘ve merged one pr https://github.com/apache/flink-cdc/pull/3118/files and introduced one more package, can you rebase master and include it? |
Make sense, updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xleoken Thanks for the PR! LGTM.
@xleoken Could you open another PR to backport the patch to release-3.0 branch? Thanks |
Thanks @PatrickRen.
We can not chang the pom.xml files in release-3.0 branch directly. |
@xleoken Ah my mistake. We don't need to backport this one to release-3.0 as 3.1 will be the first version after the donation. Thanks anyway! |
Change old com.ververica dependency to flink. Follow up #3089