Skip to content

Expose mergePolicy in KnnIndexTester #132062

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jul 29, 2025
Merged

Expose mergePolicy in KnnIndexTester #132062

merged 4 commits into from
Jul 29, 2025

Conversation

@tteofili
Copy link
Contributor

@tteofili tteofili commented Jul 29, 2025 •
edited
Loading

This makes it possible to set MergePolicy when running KnnIndexTester, via a merge_policy option.

  • "no" --> NoMergePolicy
  • "tiered" --> TieredMergePolicy
  • "log_byte" --> LogByteMergePolicy
  • "log_doc" --> LogDocMergePolicy

@elasticsearchmachine elasticsearchmachine added v9.2.0 needs:triage Requires assignment of a team area label labels Jul 29, 2025
@elasticsearchmachine elasticsearchmachine added Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch and removed needs:triage Requires assignment of a team area label labels Jul 29, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@tteofili tteofili added the priority:normal A label for assessing bug priority to be used by ES engineers label Jul 29, 2025
john-wagster
john-wagster approved these changes Jul 29, 2025
View reviewed changes
Copy link
Contributor

@john-wagster john-wagster left a comment

Choose a reason for hiding this comment

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

lgtm

@tteofili tteofili enabled auto-merge (squash) July 29, 2025 14:51
@tteofili tteofili disabled auto-merge July 29, 2025 15:21
@tteofili tteofili merged commit e488f7c into elastic:main Jul 29, 2025
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@john-wagster john-wagster john-wagster approved these changes

@iverase iverase Awaiting requested review from iverase

@carlosdelest carlosdelest Awaiting requested review from carlosdelest

Assignees

No one assigned

Labels

>non-issue priority:normal A label for assessing bug priority to be used by ES engineers :Search Relevance/Vectors Vector search Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.2.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tteofili @elasticsearchmachine @john-wagster