rmuir commented on PR #12129: URL: https://github.com/apache/lucene/pull/12129#issuecomment-1417981491
btw as followup we should investigate fixing the SortedSetDocValuesRangeQuery to use ScorerSupplier too. It is bad to do a bunch of costly `lookupTerm` operations per-segment that may be unnecessary, just to expose `cost()` (see IndexOrDocValues). Even if its only 2 lookups: we can think of these lookups as binary-searching the entire terms dict, causing random accesses, page faults, etc. -- 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
