Replies: 1 comment 1 reply
-
The original Zstd implementation is more battle tested and performant, we will NOT replace it with a pure Go version in near future, because we compress/decompress the block in one shot, the CGO is not the problem here. In the case that when CGO is not available, we may switch to the pure Go version using build tags. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
What would you like to be added:
Replace datadog's zstd with Klauspost's zstd
https://github.com/DataDog/zstd
https://github.com/klauspost/compress
Why is this needed:
Klauspost's package are pure Go and without use of "unsafe", highly maintained and optimized and also provide support for other compression methods. BadgerDB made the same move: dgraph-io/badger#1709
Beta Was this translation helpful? Give feedback.
All reactions