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-37200]Add e2e test for ElasticSearch Sink. #3898

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wangqinghuan
Copy link
Contributor

No description provided.

@wangqinghuan
Copy link
Contributor Author

wangqinghuan commented Jan 28, 2025

There is another open pr(#3535) but it's not updated. So I created a new PR to i mplement e2e test for Elasticsearch Sink. Could you review it when you have time? @leonardBang @lvyanquan

Copy link
Contributor

@yuxiqian yuxiqian left a comment

Choose a reason for hiding this comment

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

Thanks for your great contribution! Just left some general comments to make this E2E test more exhaustive.

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