jimczi commented on issue #715: LUCENE-7714 Add a range query that takes advantage of index sorting. URL: https://github.com/apache/lucene-solr/pull/715#issuecomment-502012742 We discussed offline with Julie and agreed that it doesn't really make sense to add this optimization in `IndexOrDocValuesQuery`. If range queries are fast with docvalues on a field that is used to sort the index then it will become useless to also index the field as a point. For multi-valued fields points will remain the best solution since the optimization on docvalues cannot work in this case but it should be enough to introduce the optim in `SortedNumericDocValuesField#newSlowRangeQuery` and `NumericDocValuesField#newSlowRangeQuery` or in its own query if we don't want to mix. > It is now possible that match and explain will be slower for SortedNumericDocValuesRangeQuery, since we may perform a binary search instead of just checking a single document value. Perhaps I should override these methods and ensure we always use the old approach? Agreed, the old approach makes more sense for these cases that advance to a specific document directly.
---------------------------------------------------------------- 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: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
