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

Reply via email to