Skip to content

Conversation

@15zed
Copy link

@15zed 15zed commented Dec 4, 2025

What is the theme of this PR?

add elasticsearch connector to the apache-geaflow project

Which issue is it related to?

it is ralated to #57

What changes did I make?

  • Tests have Added for the changes
  • Add elastic dependency to the pom.xml
  • Add elasticConfigkeys、elasticTableConnetor、elasticTableSink etc. files to complete this function

@kitalkuyo-gita
Copy link
Contributor

kitalkuyo-gita commented Dec 4, 2025

@15zed Hello, thank you for your contribution. Please resolve the conflict with the main branch first. Since the Elasticsearch PR was merged yesterday, there are still some tasks to be optimized. Would you be interested in resolving them?

Copy link
Author

@15zed 15zed left a comment

Choose a reason for hiding this comment

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

I have resolved the conflicts, what need to consider is the version difference of elasticsearch, there are 7.17.9 and 7.17.10, its compatible.

<groupId>org.elasticsearch.client</groupId>
<artifactId>elasticsearch-rest-high-level-client</artifactId>
<version>${elasticsearch.version}</version>
<version>7.17.9</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use ${...} to explicitly specify the version.

Copy link
Author

Choose a reason for hiding this comment

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

ok,done.

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.

2 participants