-
Notifications
You must be signed in to change notification settings - Fork 25.3k
[ML] Update inference api rest spec #124151
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
[ML] Update inference api rest spec #124151
Conversation
Pinging @elastic/ml-core (Team:ML) |
Docs changes: #124157 |
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
"inference.stream_completion": { | ||
"documentation": { | ||
"url": "https://www.elastic.co/guide/en/elasticsearch/reference/master/post-stream-inference-api.html", | ||
"description": "Perform streaming inference" |
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.
"description": "Perform streaming inference" | |
"description": "Perform streaming completion inference" |
💔 Backport failed
You can use sqren/backport to manually backport by running |
* Pulling api spec changes * Fixing test and updating code javadoc
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
* Pulling api spec changes * Fixing test and updating code javadoc (cherry picked from commit 3a472eb) # Conflicts: # x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test/inference/inference_crud.yml
* Pulling api spec changes * Fixing test and updating code javadoc (cherry picked from commit 3a472eb) # Conflicts: # x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test/inference/inference_crud.yml
* Pulling api spec changes * Fixing test and updating code javadoc
* Pulling api spec changes * Fixing test and updating code javadoc
This PR updates the rest api spec for the inference APIs to be inline with the changes that are going in here: elastic/elasticsearch-specification#3545
This PR needs to be merged first before the elasticsearch-specification PR can be merged.
The docs might need to be updated in a few places but I'll need to do that directly to 8.x and potential in main separately.