Skip to content

Conversation

@thecoop
Copy link
Member

@thecoop thecoop commented Oct 15, 2025

Create classes for quantized int vector formats that support bfloat16 and direct IO. This uses the new Lucene104 quantizer.

@elasticsearchmachine elasticsearchmachine added v9.3.0 Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch labels Oct 15, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@thecoop thecoop changed the base branch from main to lucene_snapshot October 23, 2025 08:52
@thecoop thecoop requested a review from benwtrent October 23, 2025 14:05
@thecoop thecoop requested review from a team as code owners October 23, 2025 15:20
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 need to remember why I started that Flat vs Format pattern for bit vectors that you have continued here, I am not sure its actually warranted.

Comment on lines 70 to 71
return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer) {
};
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer) {
};
return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer);

Copy link
Member Author

@thecoop thecoop Oct 24, 2025

Choose a reason for hiding this comment

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

The constructor is protected, so it needs to be called from a sneaky subclass. See apache/lucene#15357

@thecoop
Copy link
Member Author

thecoop commented Oct 24, 2025

I've removed the intermediate format, and pushed everything into the top-level formats

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

Labels

>non-issue :Search Relevance/Vectors Vector search Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants