-
Notifications
You must be signed in to change notification settings - Fork 65
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
Split get_all
variants out of ml.get_memory
and ml.get_message
#796
Conversation
e456a4e
to
eb0916f
Compare
Changes AnalysisCommit SHA: 30f5235 API ChangesSummary
ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12798109767/artifacts/2437584758 API Coverage
|
83c32a4
to
a7c3851
Compare
Spec Test Coverage Analysis
|
status: [200, 404] | ||
parameters: | ||
memory_id: ${create_memory.memory_id} | ||
chapters: |
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.
This file should be memory/messages.yaml
.
Signed-off-by: Thomas Farr <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
d03c1b3
to
aedaf8a
Compare
Signed-off-by: Thomas Farr <[email protected]>
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.
Naming of files is still off, but I can deal with this later as part of what I am working on.
Description
Split
get_all
variants out ofml.get_memory
andml.get_message
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.