jpountz commented on PR #14401:
URL: https://github.com/apache/lucene/pull/14401#issuecomment-2759494063

   > This would also benefit https://github.com/apache/lucene/pull/14273
   
   I don't think so, or rather taking advantage of range collection shouldn't 
help more than what #14273 does with `RangeDocIdStream`?
   
   For clarity, this `collectRange` method is more useful for aggregating 
values (facet "recorders" if I use the terminology of the new faceting 
framework?). Implementations would need to consult the `DocValuesSkipper` to 
check if it has pre-aggregated data over ranges of doc IDs that are contained 
in the range of doc IDs passed to `collectRange`. These sub ranges could be 
aggregated in constant time, without having to iterate over docs.


-- 
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