-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Upgrade Elasticsearch client to 7.x #20248
Conversation
ba5b2b4
to
ebaa370
Compare
This removes support for Elasticsearch 6.x while introducing support and test coverage for 8.x. Both http security and user authentication are enabled by default.
ebaa370
to
e604886
Compare
...rino-elasticsearch/src/test/java/io/trino/plugin/elasticsearch/ElasticsearchQueryRunner.java
Show resolved
Hide resolved
...rino-elasticsearch/src/test/java/io/trino/plugin/elasticsearch/ElasticsearchQueryRunner.java
Show resolved
Hide resolved
It's rather cool, @wendigo. So, far it fails on Opensearch and Ngnix. |
@mkhludnev yeah, i'm aware. Misconfiguration |
Hold on, before get into this I encounter #11377, now it seems like it's mandatory. Will Opensearch connector be a separate plugin or |
See #11295 (comment) |
OpenSearch support is moved to a separate connector via #20257 |
Release notes:
Add support for Elasticsearch 8.x and drop support for Elasticsearch 6.x