Skip to content

Conversation

tteofili
Copy link
Contributor

@tteofili tteofili commented May 22, 2025

This allows dense_vector fields to be updated to bbq_flat / bbq_hnsw.
Allowed update paths:

  • *_flat --> bbq_flat/bbq_hnsw
  • *_hnsw --> bbq_hnsw

@tteofili tteofili marked this pull request as ready for review June 10, 2025 10:39
@tteofili tteofili added :Search Relevance/Vectors Vector search auto-backport Automatically create backport pull requests when merged labels Jun 10, 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.

@elasticsearchmachine elasticsearchmachine added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jun 10, 2025
@benwtrent benwtrent self-requested a review June 10, 2025 14:45
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.

Other than the one comment this lgtm!

@tteofili tteofili removed the auto-backport Automatically create backport pull requests when merged label Jun 10, 2025
@tteofili tteofili added the auto-backport Automatically create backport pull requests when merged label Jun 10, 2025
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 this is good.

One question, what happens if the index was created with dims less than our minimum BBQ dims?

Do we fail the upgrade or do we get stuck in a bad state?

@tteofili
Copy link
Contributor Author

tteofili commented Jun 16, 2025

One question, what happens if the index was created with dims less than our minimum BBQ dims?

Do we fail the upgrade or do we get stuck in a bad state?

if you have the default (e.g., you specify nothing in index_options), the upgrade will keep the default, hence it will start failing if/when the new default becomes bbq_hnsw.
currently it doesn't because if you change nothing it will keep the default (int8_hnsw).
but if, during upgrade, you also update the mappings and specify index_options with bbq, it will fail.

@tteofili tteofili merged commit 629a366 into elastic:main Jun 16, 2025
17 of 18 checks passed
@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 128291

@tteofili
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

Questions ?

Please refer to the Backport tool documentation

tteofili added a commit to tteofili/elasticsearch that referenced this pull request Jun 16, 2025
@shainaraskas shainaraskas added the docs-missing-applies-tags PRs that are missing docs applies_to tags for an upcoming release. label Jul 22, 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 docs-missing-applies-tags PRs that are missing docs applies_to tags for an upcoming release. >enhancement :Search Relevance/Vectors Vector search 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.

7 participants