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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
