jpountz commented on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999361666


   FYI @gsmiller and I looked into using a FOR-like encoding for doc values at 
https://issues.apache.org/jira/browse/LUCENE-10033. In short, this made queries 
that look at most docs like the `Browse*` faceting tasks much faster, but the 
queries that only look at a subset of the index like `HighTermMonthSort` much 
slower.
   
   I like this change, it's a net win that is not only going to speed up binary 
doc values but also sorted set and sorted numeric doc values, which also use a 
direct monotonic reader to map doc IDs to value indexes? 
`DirectMonotonicReader#binarySearch` uses a similar idea to avoid doing the 
same operation over and over again. Let's update this pull request to limit the 
impact to the API to DirectMonotonicReader, since it's the only LongValues impl 
where we ever need to read two consecutive values at the same time?


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

Reply via email to