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

[FLINK-25568] Support Elasticsearch Source Connector #62

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

Conversation

kaori-seasons
Copy link

According to FLIP-127 to achieve source read function

@boring-cyborg
Copy link

boring-cyborg bot commented May 10, 2023

Thanks for opening this pull request! Please check out our contributing guidelines. (https://flink.apache.org/contributing/how-to-contribute.html)

@WenDing-Y
Copy link

Support for FilterPushDown, ProjectPushDown, and LimitPushDown to optimize query performance. Have these functions been implemented? I don't seem to have seen this part of the code

@kaori-seasons
Copy link
Author

kaori-seasons commented May 12, 2023

Support for FilterPushDown, ProjectPushDown, and LimitPushDown to optimize query performance. Have these functions been implemented? I don't seem to have seen this part of the code

Hello, I am very happy to read your comments. Currently this PR belongs to draft status. For the three push-downs you mentioned. As shown in FLIP-127, I am going to implement related implementations of FilterPushDown, ProjectPushDown, and LimitPushDown and optimize es query expressions. Currently, only the ProjectPushDown interface is implemented, please refer to Elasticsearch7DynamicSource#applyProjection.

Due to busy work, I will finish the development after a while.

@MartijnVisser
Copy link
Contributor

@kaori-seasons Are you still planning to work on this PR?

@kaori-seasons kaori-seasons marked this pull request as ready for review December 8, 2023 23:52
@kaori-seasons
Copy link
Author

@kaori-seasons Are you still planning to work on this PR?
@MartijnVisser
Hello, I just saw your reply recently. Due to work, I did not continue this Pull request. I will start to resolve the conflict this month and request a review.

# Conflicts:
#	flink-connector-elasticsearch-base/pom.xml
#	flink-connector-elasticsearch-base/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/ElasticsearchApiCallBridge.java
#	flink-connector-elasticsearch-base/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/table/ElasticsearchRowDataLookupFunction.java
#	flink-connector-elasticsearch-base/src/test/java/org/apache/flink/streaming/connectors/elasticsearch/ElasticsearchSinkBaseTest.java
#	flink-connector-elasticsearch6/src/main/java/org/apache/flink/streaming/connectors/elasticsearch6/Elasticsearch6ApiCallBridge.java
#	flink-connector-elasticsearch7/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/table/Elasticsearch7DynamicSource.java
#	flink-connector-elasticsearch7/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/table/Elasticsearch7DynamicTableFactory.java
#	flink-connector-elasticsearch7/src/main/java/org/apache/flink/streaming/connectors/elasticsearch7/Elasticsearch7ApiCallBridge.java
#	flink-connector-elasticsearch7/src/test/java/org/apache/flink/streaming/connectors/elasticsearch/table/Elasticsearch7DynamicSinkITCase.java
#	flink-connector-elasticsearch7/src/test/java/org/apache/flink/streaming/connectors/elasticsearch/table/Elasticsearch7DynamicTableFactoryTest.java
#	pom.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants