jpountz commented on PR #12405:
URL: https://github.com/apache/lucene/pull/12405#issuecomment-1632507845
Thanks for adding the enum. In my view, we now need the two following
changes:
- `isMissingValueCompetitive()` should return false if the missing value is
equal to the bottom value and the pruning is SKIP_MORE
- The competitive iterator can better tune the min/max values. For instance
currently, if the bottom value is 5 and the sort is ascending, we'll use a
range on [MIN_VALUE, 5] to filter competitive hits. But with SKIP_MORE, we
could now make it [MIN_VALUE, 4]. FYI we have `NumericUtils#(add|subtract)` to
add/subtract on binary representations of numbers.
--
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
