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

Update language around client version compatibility #6939

Merged
merged 6 commits into from
Apr 11, 2024
Merged

Conversation

hdhalter
Copy link
Contributor

Description

Addresses community feedback from Slack.

Issues Resolved

See https://opensearch.slack.com/archives/C052GAV1MQF/p1712689014138299

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@hdhalter hdhalter added the 3 - Tech review PR: Tech review in progress label Apr 10, 2024
@hdhalter hdhalter added the backport 2.13 PR: Backport label for 2.13 label Apr 10, 2024
Copy link
Contributor

@nhtruong nhtruong left a comment

Choose a reason for hiding this comment

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

Looks good!

@hdhalter hdhalter added 4 - Doc review PR: Doc review in progress and removed 3 - Tech review PR: Tech review in progress labels Apr 11, 2024
_clients/index.md Outdated Show resolved Hide resolved
Most clients that work with Elasticsearch OSS 7.10.2 *should* work with OpenSearch, but the latest versions of those clients might include license or version checks that artificially break compatibility. This page includes recommendations around which versions of those clients to use for best compatibility with OpenSearch.
Most clients that work with ElasticSearch OSS 7.10.2 *should* work with OpenSearch 1.x, but the latest versions of those clients might include license or version checks that artificially break compatibility. The table that follows includes recommendations for which client versions to use for best compatibility with OpenSearch 1.x. From OpenSearch 2.0 and later, no ElasticSearch clients are fully compatible with OpenSearch.

While OpenSearch and ElasticSearch share a majority of core features, mixing and matching the client and server has a very high risk of errors and unexpected results. The issues will continue to get worse over time as OpenSearch and ElasticSearch diverge even further from each other. So, even if your ElasticSearch client still works with your OpenSearch cluster, we highly recommend using OpenSearch clients for OpenSearch clusters.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
While OpenSearch and ElasticSearch share a majority of core features, mixing and matching the client and server has a very high risk of errors and unexpected results. The issues will continue to get worse over time as OpenSearch and ElasticSearch diverge even further from each other. So, even if your ElasticSearch client still works with your OpenSearch cluster, we highly recommend using OpenSearch clients for OpenSearch clusters.
While OpenSearch and Elasticsearch share a majority of core features, mixing and matching the client and server has a very high risk of errors and unexpected results. The issues will continue to get worse over time as OpenSearch and Elasticsearch diverge even further from each other. So, even if your Elasticsearch client still works with your OpenSearch cluster, we highly recommend using OpenSearch clients for OpenSearch clusters.

Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

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

Please see the suggested edits to tighten or clarify writing. To avoid duplicative efforts, has this been scheduled for editorial review?

_clients/index.md Outdated Show resolved Hide resolved
## Legacy clients

Most clients that work with Elasticsearch OSS 7.10.2 *should* work with OpenSearch, but the latest versions of those clients might include license or version checks that artificially break compatibility. This page includes recommendations around which versions of those clients to use for best compatibility with OpenSearch.
Most clients that work with ElasticSearch OSS 7.10.2 *should* work with OpenSearch 1.x, but the latest versions of those clients might include license or version checks that artificially break compatibility. The table that follows includes recommendations for which client versions to use for best compatibility with OpenSearch 1.x. From OpenSearch 2.0 and later, no ElasticSearch clients are fully compatible with OpenSearch.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Most clients that work with ElasticSearch OSS 7.10.2 *should* work with OpenSearch 1.x, but the latest versions of those clients might include license or version checks that artificially break compatibility. The table that follows includes recommendations for which client versions to use for best compatibility with OpenSearch 1.x. From OpenSearch 2.0 and later, no ElasticSearch clients are fully compatible with OpenSearch.
Clients that work with ElasticSearch OSS 7.10.2 should work with OpenSearch 1.x. The latest versions of those clients, however, might include license or version checks that artificially break compatibility. The following table provides recommendations for which client versions to use for best compatibility with OpenSearch 1.x. For OpenSearch 2.0 and later, no Elasticsearch clients are fully compatible with OpenSearch.

_clients/index.md Outdated Show resolved Hide resolved
_clients/index.md Outdated Show resolved Hide resolved
hdhalter and others added 4 commits April 11, 2024 08:45
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Copy link
Contributor Author

@hdhalter hdhalter left a comment

Choose a reason for hiding this comment

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

@vagimeli - Thank you for the review. I may have overwritten one of your comments by accepting a previous one. Can you please review this one comment and accept if this looks good? Thanks!

_clients/index.md Outdated Show resolved Hide resolved
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
@hdhalter hdhalter merged commit b7d50c5 into main Apr 11, 2024
6 checks passed
@github-actions github-actions bot deleted the hdhalter-patch-1 branch April 11, 2024 16:26
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 11, 2024
* Update language around client version compatibility

Signed-off-by: Heather Halter <[email protected]>

* Update _clients/index.md

Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _clients/index.md

Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _clients/index.md

Co-authored-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _clients/index.md

Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _clients/index.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Heather Halter <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
(cherry picked from commit b7d50c5)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - Doc review PR: Doc review in progress backport 2.13 PR: Backport label for 2.13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants