pull request to please lint gods

[varun06]

No description provided.

[bai]

[varun06]

ha, exactly. @bai are we planning to cut a release soon. we have bunch of commits?

[bai]

Yeah, I want to cut is asap. One thing that I wanted to include is support for zstd 1.3.8. Kafka 2.2.0 uses zstd 1.3.8, while we use DataDog's zstd 1.3.5.

If my zstd PR doesn't go through today, I'll compile changelog + make a release tomorrow.

[valyala]

PTAL https://github.com/valyala/gozstd , which includes support for zstd 1.3.8 .

[bai]

@valyala I bumped zstd dep in #1350. Out of curiosity, how's your implementation different from DataDogs?

[valyala]

gozstd has the following features comparing to DataDog's zstd:

• It vendors upstream zstd without any modifications in the zstd folder. The update process for the upstream zstd is easy - just run make update-zstd from the root of the repository. For example, I just updated the upstream zstd to v1.4.0 in this commit.
• It is optimized for speed. The API may be easily used in zero allocations mode.
• It provides faster and more efficient stream compression. It provides proper Writer.Flush call for network apps.
• It supports dictionaries.

[bai]

Thanks @valyala, will take a look!