shubhamvishu commented on issue #15057: URL: https://github.com/apache/lucene/issues/15057#issuecomment-3189035596
@msokolov @benwtrent I think its a miss because when [bulk scoring in HnswGraphSearcher](https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraphSearcher.java#L325-L341) we do respect the visitedLimit and only score `visitLimit - visitCount` no. of docs ([code](https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraphSearcher.java#L339C60-L339C70)) but not in the exhautive search case. I wonder if doing the same is the right way here? WDYT? > seems as if we should fix the test to be aware of bulk scoring and round up? Is there a way to know how many will be scored in bulk? Right now we bulk score 64 docs but that is not exposed. -- 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
