gtroitskiy commented on a change in pull request #217: URL: https://github.com/apache/lucene/pull/217#discussion_r676822527
########## File path: lucene/facet/src/java/org/apache/lucene/facet/DrillSidewaysScorer.java ########## @@ -195,11 +195,8 @@ private void doQueryFirstScoring(Bits acceptDocs, LeafCollector collector, DocsA collectDocID = docID; Review comment: sorry, I've been away for a while thank you for the sketch, your approach is definitely more elegant :slightly_smiling_face: except I'm not sure we need caching, since by design only one drill-down collector is being called for a specific docId PS force-pushed changes -- 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