Add zstd compression support

[martinabeleda]

Motivation

See #1236

Solution

Add zstd as a CompressionEncoding and as a feature/optional dependency.

New to rust so looking for some feedback on my approach to testing.

[h33p]

Not a hyper dev, but approach seems sensible and extensible. I would look into the function visibility comment, though.

[mpetri]

@martinabeleda any updates on this CR?

Are you waiting for feedback from the actual maintainers? @LucioFranco would you be willing to accept this change? I think this will lead to substantial performance improvement for especially compressible messages.

[martinabeleda]

@martinabeleda any updates on this CR?

Are you waiting for feedback from the actual maintainers? @LucioFranco would you be willing to accept this change? I think this will lead to substantial performance improvement for especially compressible messages.

@mpetri I just have a few of your comments to address but other than that I'm waiting for maintainers. I'll have some time to finish the PR later today so can try to reach @LucioFranco on discord

[LucioFranco]

Hi sorry for the delay. I think the comments make sense but I am taking a small break for the next few weeks so the progress has been slow here.

[mpetri]

Overall this looks good now. One minor comment on the default value of the compressor

[QuentinPerez]

Hi @LucioFranco,

I have tested it in production, and it works perfectly! Thank you, @martinabeleda! When do you think we can merge it? I don't want to stay on a fork with that patch forever.

[LucioFranco]

@martinabeleda seems like there are some failures around feature flags in CI.

[QuentinPerez]

@martinabeleda do you have the time to fix the CI issue ? Otherwise I will re-open the PR with your code and the fixes for the CI.

[martinabeleda]

@martinabeleda do you have the time to fix the CI issue ? Otherwise I will re-open the PR with your code and the fixes for the CI.

@QuentinPerez I haven't had time to work on the fixes so would appreciate if you could help fix CI. Thanks!!

[tottoto]

Resolved in #1532. Thank you!