romseygeek commented on issue #12230: URL: https://github.com/apache/lucene/issues/12230#issuecomment-1510963485
I'm not sure adding this to `TotalHitCountCollector` makes sense - surely the whole point of this collector is that you want an accurate hit count? The MultiCollector javadocs already call out that mixing collectors that can early terminate with ones that can't is likely to give you bad performance, and recommend running two searches instead. -- 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