mikemccand / luceneutil

Various utility scripts for running Lucene performance tests
Apache License 2.0
202 stars 113 forks source link

Does HNSW search sometimes match NO_MORE_DOCS? #297

Open vigyasharma opened 2 weeks ago

vigyasharma commented 2 weeks ago

We have a check to separately handle NO_MORE_DOCS while parsing the results from an HNSW approximate search. We should not be returning NO_MORE_DOCS in the first place. Creating an issue to investigate when and why this happens.

https://github.com/mikemccand/luceneutil/blob/main/src/main/knn/KnnGraphTester.java#L619-L624

mikemccand commented 18 hours ago

I'm hoping this was a holdover from some long ago fixed bug? I'll try simply removing that check and see if anything bad happens...