Skip to content

Conversation

@kderusso
Copy link
Member

@kderusso kderusso commented Oct 1, 2025

As elastic/elasticsearch#119011 adds support for knn queries over semantic_text fields where inference ID may be inferred, this PR fixes an issue where it was required.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2025

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

API Status Request Response
search 🔴 2533/2602 → 2540/2602 2602/2602

You can validate this API 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.

Added the 9.2 backport

@pquentin pquentin merged commit 1560bbb into main Oct 2, 2025
17 of 18 checks passed
@pquentin pquentin deleted the kderusso/semantic-optional-inference branch October 2, 2025 06:55
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
pquentin pushed a commit that referenced this pull request Oct 2, 2025
pquentin pushed a commit that referenced this pull request Oct 2, 2025
pquentin pushed a commit that referenced this pull request Oct 2, 2025
pquentin pushed a commit that referenced this pull request Oct 2, 2025
pquentin pushed a commit that referenced this pull request Oct 2, 2025
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