valyala / gozstd

go wrapper for zstd
MIT License
420 stars 60 forks source link

Update to Zstd 1.5.0 #35

Closed jimsmart closed 3 years ago

jimsmart commented 3 years ago

Hi,

I see Zstd 1.5.0 has just been released, with improved performance.

https://github.com/facebook/zstd/releases/tag/v1.5.0

I tried running make update-zstd in a copy of the gozstd repo - to submit a pull request - but when I committed the changes afterwards, there was only a single file that had changed. It didn't seem to have updated. I suspect there is more to this process?

Please update gozstd to use Zstd 1.5.0 - thanks!

valyala commented 3 years ago

gozstd has been updated to zstd v1.5.0 in the commit https://github.com/valyala/gozstd/commit/f86d73fc96fedcf0d78ed906e414feaff7115018

valyala commented 3 years ago

This commit has been included in gozstd v1.10.0. Closing the feature request as done.

jimsmart commented 3 years ago

Thanks very much, appreciated!