Skip to content

DiskBBQ - missing min competitive similarity check on tail docs#135851

Merged
tteofili merged 2 commits intoelastic:mainfrom
tteofili:dbbq_miss_sim_check
Oct 3, 2025
Merged

DiskBBQ - missing min competitive similarity check on tail docs#135851
tteofili merged 2 commits intoelastic:mainfrom
tteofili:dbbq_miss_sim_check

Conversation

@tteofili
Copy link
Contributor

@tteofili tteofili commented Oct 2, 2025

This looks like a missing check on minimum competitive similarity for the KnnCollector to collect docs.

@tteofili tteofili requested a review from benwtrent October 2, 2025 14:27
@elasticsearchmachine elasticsearchmachine added Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.0 labels Oct 2, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@benwtrent benwtrent left a comment

Choose a reason for hiding this comment

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

I think the collector does this internally, but this is good.

@tteofili tteofili merged commit c2cdc0a into elastic:main Oct 3, 2025
30 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Search Relevance/Vectors Vector search Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.2.0 v9.3.0

3 participants