Add zlib-ng?
bjornharrtell opened this issue · 4 comments
Feature request :)
zlib-ng does seem like a reasonable drop-in replacement - provided it can indeed be made to work for the other consumers of it (i.e. curl and openssl - libpq currently doesn't even compile with zlib..).
Long term I am kind of hoping to have less C dependencies here because of maintenance required, but that requires rustls to iron out some of their last production limiting issues so that means at least some years more of maintaining openssl here.
So.. maybe? I don't really know what benefits this would entail for users of a docker image? I honestly wasn't expecting much movement on zlib at this point, so there probably would have to be some good performance benefits for it to be worth the maintenance overhead.
Unfortunately the pure rust impl performs worse than both the classic c zlib and zlib-ng is significatly faster than zlib.
Do you know of any benchmarks around to look at?
There are bits and pieces around, fx. Frommi/miniz_oxide#5. But I see there is stuff like https://lib.rs/crates/simd-adler32.. but seems it is not easily opted in.