mikemccand commented on issue #12696: URL: https://github.com/apache/lucene/issues/12696#issuecomment-1786949852
Thanks for testing @jpountz. I think at some point we also enabled patching for the freq blocks inside `.doc` file too? Normally the `IntNRQ` (1D points numeric range query) is very noisy, but maybe this gain is real? p-value seems to think it could be close to real? The conjunctive and disjunctive gains are awesome. > Regarding backward compatibility, let's do it with codecs as usual: fork `Lucene90PostingsFormat` into a new `Lucene99PostingsFormat` that uses PFOR for postings. Then the codec infrastructure will make sure to keep using the old postings format for existing segments and the new postings format for new segments (including merged ones). +1 > I have a preference for keeping PFOR for positions and only moving postings to FOR (essentially reverting https://github.com/apache/lucene/pull/69). +1 This change might make SIMD decoding more palatable for the unpached `int[]` blocks (doc, freq). -- 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
