romseygeek commented on code in PR #16166:
URL: https://github.com/apache/lucene/pull/16166#discussion_r3354629397
##########
lucene/core/src/java/org/apache/lucene/search/DocValuesRangeIterator.java:
##########
@@ -276,6 +285,9 @@ private static final class DocValuesBlockRangeIterator
extends DocValuesRangeIte
// Non-null only for single-valued numeric doc values, in which case a
whole block can be
// range-evaluated in one shot in intoBitSet rather than confirming
matches() one doc at a time.
private final NumericDocValues numericValues;
+ // Non-null only for multi-valued sorted-numeric doc values. It enables
the same block-level
Review Comment:
This is beginning to get a bit unwieldy - can we add some private
BulkXXXRangeIterator subclasses that implement `intoBitSet` separately?
--
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]
