-
Notifications
You must be signed in to change notification settings - Fork 62
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 Cat API query descriptions. #726
Update Cat API query descriptions. #726
Conversation
Signed-off-by: Archer <naarcha@amazon.com>
Changes AnalysisCommit SHA: 24682f0 API ChangesSummary
ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12283607679/artifacts/2307876719 API Coverage
|
Spec Test Coverage Analysis
|
Signed-off-by: Archer <naarcha@amazon.com>
Signed-off-by: Archer <naarcha@amazon.com>
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.
Please apply changes globally to all instances of the same descriptions. Made suggestions in opensearch-project/documentation-website#8930. cc: @natebower
Note that it's possible to consolidate schema and use |
Is that something we should do now or flag it for later? For the time being, we could do a simple find and replace to make the descriptions consistent. |
$ref is definitely the preferred long-term strategy. |
It's really up to you. I would try to consolidate those into $refs if you're going to do a lot of replacing. |
Can I merge this? |
Lets go ahead and merge this one. I'll address the refs in a separate PR. |
Updates the the query parameter descriptions for three APIs, Cat Aliases, Cat Allocation, and Cat Cluster Manager.
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.