Skip to content

Conversation

@thecoop
Copy link
Member

@thecoop thecoop commented Jun 12, 2025

Check the positions on multi-phrase queries as well as simple ones

Resolves #123871

@thecoop thecoop added >bug auto-backport Automatically create backport pull requests when merged :Search Relevance/Search Catch all for Search Relevance v8.19.0 v9.1.0 labels Jun 12, 2025
@elasticsearchmachine
Copy link
Collaborator

Hi @thecoop, I've created a changelog YAML for you.

@thecoop thecoop marked this pull request as ready for review June 12, 2025 12:19
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

@elasticsearchmachine elasticsearchmachine added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jun 12, 2025
Copy link
Contributor

@iverase iverase left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@carlosdelest carlosdelest left a comment

Choose a reason for hiding this comment

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

Thanks @thecoop !

@thecoop thecoop merged commit 3988ee1 into elastic:main Jun 12, 2025
18 checks passed
@thecoop thecoop deleted the multi-match-error-status branch June 12, 2025 15:05
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.19 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 129326

thecoop added a commit to thecoop/elasticsearch that referenced this pull request Jun 12, 2025
Check positions on MultiPhraseQueries as well as phrase queries (elastic#129326)
thecoop added a commit that referenced this pull request Jun 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >bug :Search Relevance/Search Catch all for Search Relevance Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

500 error in multi match query for field without position data

5 participants