RamakrishnaChilaka opened a new pull request, #15174: URL: https://github.com/apache/lucene/pull/15174
The previous code compared numBitsNextBitsPerValue ≤ docRange to decide whether to store the doc-delta block with PackedInts or as a bit-set. docRange is the number of documents, not the number of bits the bit-set would occupy, so the test did not reflect the actual space that would be written. ### Change Replace the comparison with `numBitsNextBitsPerValue ≤ numBitSetLongs * Long.SIZE` so the decision is based on the exact bit footprint of each alternative. The bit-set path is now taken only when it is strictly smaller on disk. We can skip change-log as it is a marginal change, I will add it, if you prefer -- 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]
