Skip to content

[DOCS] Fix bbq memory size estimate #123453

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
Closed
wants to merge 1 commit into from
Closed

[DOCS] Fix bbq memory size estimate #123453

wants to merge 1 commit into from

Conversation

weizijun
Copy link
Contributor

If I understand correctly, the memory size of bbq is num_vectors * (num_dimensions/8 + 14)

14 bits consisting of 3 floats and a short value.

@github-actions GitHub Actions
Copy link
Contributor

Documentation preview:

@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.

@elasticsearchmachine elasticsearchmachine added v9.1.0 needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Feb 26, 2025
@weizijun
Copy link
Contributor Author

@benwtrent can you help to see the PR.

@kingherc kingherc added :Search Relevance/Vectors Vector search >docs General docs changes labels Mar 4, 2025
@elasticsearchmachine elasticsearchmachine added Team:Docs Meta label for docs team Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch and removed needs:triage Requires assignment of a team area label labels Mar 4, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Collaborator

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

@benwtrent benwtrent self-assigned this Mar 4, 2025
benwtrent
benwtrent approved these changes Mar 4, 2025
View reviewed changes
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.

looks good!

@benwtrent
Copy link
Member

@weizijun Could you open this PR against 8.x ? I can handle the v9+ versions in another PR as things regarding docs are being migrated around.

@weizijun
Copy link
Contributor Author

weizijun commented Mar 4, 2025

@weizijun Could you open this PR against 8.x ? I can handle the v9+ versions in another PR as things regarding docs are being migrated around.

Let me see.

@weizijun weizijun mentioned this pull request Mar 4, 2025
Merged
@weizijun
Copy link
Contributor Author

weizijun commented Mar 4, 2025

@weizijun Could you open this PR against 8.x ? I can handle the v9+ versions in another PR as things regarding docs are being migrated around.

@benwtrent I open the PR : #124022

@leemthompo
Copy link
Contributor

@weizijun FYI there's some context about the docs migration and workflow changes: https://github.com/elastic/elasticsearch/tree/main/docs#readme 🙂

@leemthompo leemthompo closed this Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benwtrent benwtrent benwtrent approved these changes

Assignees

@benwtrent benwtrent

Labels

>docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team :Search Relevance/Vectors Vector search Team:Docs Meta label for docs team Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@weizijun @elasticsearchmachine @benwtrent @leemthompo @kingherc