ChrisHegarty commented on PR #12417: URL: https://github.com/apache/lucene/pull/12417#issuecomment-1629011116
Part of the issue here is that we are (well, at least I have been), looking at bit packing and unpacking in isolation. We really need to consider the impact on the codec more holistically - including prefix sum (which we've not done so far). It's silly to represent the decoded values as int[], and then coerce then into long[] for prefix sum - we should just vectorise prefix sum to work on int[] (in a reasonable way). I don't even see how this is supposed to work in this PR - have I missed something ? -- 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
