[GitHub] [lucene-solr] jpountz commented on pull request #1538: LUCENE-9368: Use readLELongs to read docIds on BKD leaf nodes

[GitBox <gi...@apache.org>]

jpountz commented on pull request #1538:
URL: https://github.com/apache/lucene-solr/pull/1538#issuecomment-635385634


   > Adding a special case when docIds are just incrementing by one as well
   
   I can't think of many cases when that would kick in, this would only be used for single-valued fields that are used for index sorting?
   
   > Yes, that was my idea as a follow up and the reason I added the 8 /16 bits so we can use it later
   
   Do you mean something like stealing a bit of the byte we use to store the number of bits per value to record whether doc IDs are offseted by some constant? If so, then maybe we should do it now so 8/16 bits per value would have a chance of getting used on real data?


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org