Skip to content

Conversation

@davidkyle
Copy link
Member

The input_type parameter is used in the text embedding Inference API.

It's documented in the general inference API docs https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-inference but not in the specific text embedding inference doc https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-text-embedding.

For this PR I've just copied and pasted the input_type details from https://github.com/elastic/elasticsearch-specification/blob/main/specification/inference/inference/InferenceRequest.ts#L97

@github-actions
Copy link
Contributor

Following you can find the validation changes against the target branch for the APIs.

No changes detected.

You can validate these APIs yourself by using the make validate target.

Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM.

According to elastic/elasticsearch#122638, this landed in 8.19 and 9.1. Adopting the corresponding backport labels.

@davidkyle davidkyle merged commit c2e75e1 into main Sep 12, 2025
12 of 14 checks passed
@davidkyle davidkyle deleted the input-type branch September 12, 2025 11:39
@davidkyle
Copy link
Member Author

Thanks for adding the backport labels @pquentin

github-actions bot pushed a commit that referenced this pull request Sep 12, 2025
github-actions bot pushed a commit that referenced this pull request Sep 12, 2025
pquentin pushed a commit that referenced this pull request Sep 12, 2025
#5274) (#5282)

(cherry picked from commit c2e75e1)

Co-authored-by: David Kyle <david.kyle@elastic.co>
pquentin pushed a commit that referenced this pull request Sep 12, 2025
#5274) (#5283)

(cherry picked from commit c2e75e1)

Co-authored-by: David Kyle <david.kyle@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants