msokolov commented on a change in pull request #1351: LUCENE-9280: Collectors 
to skip noncompetitive documents
URL: https://github.com/apache/lucene-solr/pull/1351#discussion_r396127831
 
 

 ##########
 File path: lucene/core/src/java/org/apache/lucene/search/FieldComparator.java
 ##########
 @@ -928,4 +928,9 @@ public int compareTop(int doc) throws IOException {
     @Override
     public void setScorer(Scorable scorer) {}
   }
+
+  public static abstract class IteratorSupplierComparator<T> extends 
FieldComparator<T> implements LeafFieldComparator {
+    abstract DocIdSetIterator iterator();
+    abstract void updateIterator() throws IOException;
 
 Review comment:
   The name seems to indicate that this is something that compares 
IteratorSuppliers, when in fact it is something that is a comparator that also 
supplies iterators. I'm not sure I understand yet where it fits, but given 
that, a better name might be IterableComparator?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to