Skip to content

[FLINK-37652][Connectors/Opensearch] Move to Flink to 2.0.0 #50

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

reta
Copy link
Member

@reta reta commented Apr 11, 2025

[FLINK-37652][Connectors/Opensearch] Move to Flink to 2.0.0

@reta
Copy link
Member Author

reta commented Apr 11, 2025

@snuyanzin whenever you have time to look, please

Copy link
Contributor

@snuyanzin snuyanzin left a comment

Choose a reason for hiding this comment

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

Great to see support of 2.0 coming
I left some comments

reta added 3 commits April 16, 2025 18:37
Signed-off-by: Andriy Redko <drreta@gmail.com>
Signed-off-by: Andriy Redko <drreta@gmail.com>
Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta
Copy link
Member Author

reta commented Apr 17, 2025

I left some comments

Thanks a lot, @snuyanzin ! Addressed all except spotless version bump (I think we should do it in flink-connector-parent), thank you!

{
flink: 1.20,
jdk: '8, 11, 17, 21',
branch: main
}, {
flink: 1.19.0,
flink: 1.19.2,
jdk: '8, 11, 17, 21',
branch: v1.2
}, {
Copy link
Contributor

Choose a reason for hiding this comment

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

sorry, I think I missed it:
since 1.18-SNAPSHOT was removed in this PR
how about replacing 1.18.2 (which is a couple of lines below) with 1.20.1 ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated to 1.20.1 and added 1.20-SNAPSHOT as well

@snuyanzin
Copy link
Contributor

snuyanzin commented Apr 26, 2025

thank you for addressing feedback 👍 ,
I think there is one more minor and we are good to merge

Signed-off-by: Andriy Redko <drreta@gmail.com>
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.

2 participants