- 
                Notifications
    You must be signed in to change notification settings 
- Fork 25.6k
Create int bfloat16 vector formats #136627
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.
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
base: lucene_snapshot
Are you sure you want to change the base?
Conversation
| Pinging @elastic/es-search-relevance (Team:Search Relevance) | 
        
          
                server/src/main/java/org/elasticsearch/index/codec/vectors/es93/ES93Int8FlatVectorFormat.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
      66172e3    to
    1287a0b      
    Compare
  
            
          
                ...r/src/main/java/org/elasticsearch/index/codec/vectors/ES814ScalarQuantizedVectorsFormat.java
          
            Show resolved
            Hide resolved
        
      There was a problem hiding this 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.
| return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer) { | ||
| }; | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer) { | |
| }; | |
| return new Lucene104ScalarQuantizedVectorsWriter(state, encoding, rawVectorFormat.fieldsWriter(state), flatVectorScorer); | 
There was a problem hiding this comment.
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
This reverts commit 064392f.
| I've removed the intermediate format, and pushed everything into the top-level formats | 
Create classes for quantized int vector formats that support bfloat16 and direct IO. This uses the new Lucene104 quantizer.