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

[INLONG-10644][Sort] Add the elasticsearch-base module to implement elasticsearch connector 6&7 #10647

Merged
merged 10 commits into from
Jul 25, 2024

Conversation

XiaoYou201
Copy link
Contributor

[INLONG-10644][Sort] Add the elasticsearch-base module to implement elasticsearch connector 6&7

Fixes #10644

Motivation

Add the elasticsearch-base module to implement elasticsearch connector 6&7

To easy to develop

Modifications

Add some public constants, methods, classes, interfaces, etc.

@EMsnap
Copy link
Contributor

EMsnap commented Jul 22, 2024

Please add test result on es 6 or 7

@aloyszhang
Copy link
Contributor

Plz add source information for es connector classes

@XiaoYou201
Copy link
Contributor Author

Plz add source information for es connector classes

The es connector 6/7 are base on it. So,in my opinion, it is also a part of es connector.

@XiaoYou201
Copy link
Contributor Author

Please add test result on es 6 or 7
As follow image, I implement es 6 connector and postgreSQL to ElasticSearch6 test successfully . But es6 base on this pr, so, the es6 connector pr will be submitted after this pr merged.
image
image
image

@aloyszhang
Copy link
Contributor

aloyszhang commented Jul 24, 2024

BTW, Flink-1.15 needs this feature too, would you like to work on it?

@XiaoYou201
Copy link
Contributor Author

BTW, Flink-1.15 needs this feature too, would you like to work on it?

yes, I would like to do that, it will done soon~

@EMsnap EMsnap merged commit 2d48736 into apache:master Jul 25, 2024
10 checks passed
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.

[Feature][Sort] The base common of elasticsearch connector is neccessary.
4 participants