@lemire This is ready for review

This pull request introduces 16 alerts when merging 5b86fe7 into 1e48bd0 - view on LGTM.com

new alerts:

• 16 for Implicit narrowing conversion in compound assignment

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Hi! This looks like what I need for my project. Is there a chance that this can be merged (and hopefully released on Maven central? Is there anything I can contribute to make this happen? Thank you for your time.

I resynced the PR

Thanks. Merging.

Hi again! Thank you for the quick response. I built the latest snapshot and the new code is working, but I was a bit surprised that the encoding would be that much less efficient than the integer encoder for the same input of numbers. I'm unfortunately not well-versed in the integer encoding schemes to know if this is as expected.

Input is 500,000 integer values between 0 and 1,500,000.

IntegratedIntCompressor
269 KB

LongCompressor
2,554 KB

For context, I typically do not need the extended numberspace of longs, but I sometimes must support it. To simplify my code using the LongCompressor would make things easier.

Any input greatly appreciated!

IntegratedIntCompressor is dedicated to sorted integers, while LongCompressor manages unsorted longs. There is no IntegratedLongCompressor yet.