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