-
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
Revert "[Remove] MainResponse version override cluster setting (#3031) (#3032)" #3530
Revert "[Remove] MainResponse version override cluster setting (#3031) (#3032)" #3530
Conversation
…earch-project#3031) (opensearch-project#3032)" This reverts commit 199fbb8. Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
f9a154b
to
61403f0
Compare
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.
LGTM!
@nknize thanks for this PR. |
@saratvemulapalli @alicejw-aws: Looks like https://opensearch.org/docs/latest/clients/agents-and-ingestion-tools/index/ is still not updated. |
compatibility.override_main_response_version: true is supported v2.1.0 onwards |
This reverts commit 199fbb8 which removed the
opensearch.http.override_main_response_version
cluster setting.The setting was originally added so users of legacy clients would not have to upgrade their client to begin using OpenSearch 1.x.
With 2.0 the REST API is further fragmented from legacy clients such that users are forced to upgrade to the latest OpenSearch client.
To reduce this friction OpenSearch is working on a REST API Versioning mechanism to support API compatibility with at least the legacy 7.x line; but this will not be ready until 2.2 at the earliest.
The immediate problem is that there is no OpenSearch version of Beats and Logstash and the upstream maintainers of these projects have no plans to support OpenSearch.
This has already lead to user pain regarding document type removal in beats which is alleviated by ignoring type removal in the bulk API and temporarily adding the docType endpoints back for certain REST APIs.
It is also leading to pain for users of different pipeline processor types in filebeat as these processors have explicit version checks for compatibility only with Elasticsearch.
To alleviate this pain, OpenSearch adds back the
opensearch.http.override_main_response_version
setting so users of Beats and logstash can continue to function by spoofing the OpenSearch cluster version to7.10.2
.