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

Inclusive Naming #212

Closed
wants to merge 36 commits into from
Closed

Conversation

amoghtewari
Copy link

Description

Support for Inclusive Naming

Issues Resolved

Contextual references replaced with 'Leader'

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

@amoghtewari amoghtewari requested review from a team, axeoman and deztructor as code owners October 5, 2022 13:12
@harshavamsi
Copy link
Collaborator

Hi @amoghtewari, firstly thanks for this!

A couple of points: you have to sign off you commit with -s to enable the DCO check to pass.

We would like to first deprecate non-inclusive naming in a minor release and completely remove them in a major release. This means we would still have to support those endpoints, but add the deprecated tag to them. We would like to create new APIs and replace master with cluster_manager where appropriate, see [1].

[1] opensearch-project/opensearch-clients#16

@harshavamsi
Copy link
Collaborator

Inclusive naming is now part of the 2.0.1 release. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants