Skip to content

Setting default vector_index_type to bbq_hnsw for so_vector#807

Merged
pmpailis merged 1 commit intoelastic:masterfrom
pmpailis:setting_default_index_type_to_bbq_hnsw_for_so_vector
Jul 9, 2025
Merged

Setting default vector_index_type to bbq_hnsw for so_vector#807
pmpailis merged 1 commit intoelastic:masterfrom
pmpailis:setting_default_index_type_to_bbq_hnsw_for_so_vector

Conversation

@pmpailis
Copy link
Contributor

@pmpailis pmpailis commented Jul 4, 2025

Now that bbq_hnsw is our default for vectors with >= 384 dims, we can change the so_vector track to make use of this default as well.

Related ES PR: elastic/elasticsearch#129825

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

@pmpailis pmpailis merged commit 2326cd5 into elastic:master Jul 9, 2025
13 checks passed
@esbenchmachine esbenchmachine added the backport pending Awaiting backport to stable release branch label Dec 19, 2025
@esbenchmachine
Copy link
Collaborator

@pmpailis
A backport is pending for this PR. Please add all required vX.Y version labels.

  • If it is intended for the current Elasticsearch release version, apply the corresponding version label.
  • If it also supports past released versions, add those labels too.
  • If it only targets a future version, wait until that version label exists and then add it.
    (Each rally-tracks version label is created during the feature freeze of a new Elasticsearch branch).

Backporting entails:

  1. Ensure the correct version labels exist in this PR.
  2. Ensure backport PRs have backport label and are passing tests.
  3. Merge backport PRs (you can approve yourself and enable auto-merge).
  4. Remove backport pending label from this PR once all backport PRs are merged.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport pending Awaiting backport to stable release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants