msokolov commented on issue #12627:
URL: https://github.com/apache/lucene/issues/12627#issuecomment-2033044934

   I struggle to make this work though since the changes to make everything 
more typesafe have also made the interesting bits inaccessible. EG I thought of 
adding something like this:
   
   ```
           for (LeafReaderContext ctx : reader.leaves()) {
             CodecReader codecReader = (CodecReader) ctx.reader();
             HnswGraph graph = ((HnswGraphProvider) 
codecReader.getVectorReader()).getGraph(vectorField);
             if (HnswTestUtil.isFullyConnected(graph) == false) {
               // for now just bail out.
               return;
             }
           }
   ```
   
   to `BaseVector...QueryTest` however the VectorReader returned from that is 
hiding its HnswGraph behind a few layers of abstraction:
   
   class 
org.apache.lucene.codecs.perfield.PerFieldKnnVectorsFormat$FieldsReader cannot 
be cast to class org.apache.lucene.codecs.HnswGraphProvider 
(org.apache.lucene.codecs.perfield.PerFieldKnnVectorsFormat$FieldsReader and 
org.apache.lucene.codecs.HnswGraphProvider are in unnamed module of loader 
'app')
   
   not sure what the approved way to go about this would be
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to