jpountz opened a new pull request, #13968: URL: https://github.com/apache/lucene/pull/13968
In Lucene 8.4, we updated postings to work on long[] arrays internally. This allowed us to workaround the lack of explicit vectorization (auto-vectorization doesn't detect all the scenarios that we would like to handle) support in the JVM by summing up two integers in one operation for instance. With explicit vectorization now available, it looks like we can get more benefits from the ability to compare multiple intetgers in one operations than from summing up two integers in one operation. Moving back to ints helps compare 2x more integers at once vs. longs. The diff is large because of the codec dance: `Lucene912PostingsFormat` and `Lucene100Codec` moved to `lucene/backward-codecs` and a new `Lucene101PostingsFormat` is a copy of the previous `Lucene912PostingsFormat` with a move from long[] arrays to int[] arrays, and changes to the on-disk format for blocks of packed integers. Note that `DataInput#readGroupVInt` and `VectorUtilSupport#findNextGEQ` have been cleaned up to only support `int[]` and no longer `long[]`. -- 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
