-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Hotfix][ElasticsearchSink]Update the version in the docs. #4499
Conversation
@iture123 PTAL |
@iture123 @TyrantLucifer @Hisoka-X PTAL,thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The owner @iture123 of es connector has the different thoughts of this, later he will left his advice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ElasticsearchRowSerializer make an error annotation,The elasticsearch connector and the class ElasticsearchRowSerializer.java both support ElasticSearch version is >= 2.x and <= 8.x.
ok, fix error annotation. |
@TyrantLucifer @iture123 @Hisoka-X PTAL. |
who has time help review, thanks. |
merge dev into your branch |
done. |
@TyrantLucifer @iture123 @Hisoka-X PTAL again. |
Purpose of this pull request
The test found that the 6.x version cannot be supported.
![image](https://user-images.githubusercontent.com/40714172/229759578-2d31bf8b-cec2-4aeb-a94d-f220a9c10684.png)
Check list
New License Guide
release-note
.