-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proposal to use zstd instead of gzip #43
Comments
Ah! I had almost this exact thought the other day. That would be super cool -- a substantial size reduction and a speedup is a win in my book. Would love to see a PR. :) |
Cool, good to hear, will hopefully have something next week. Do you want it to support both, or just do a straight replacement? |
Replacement is totally fine; no need for both when the format itself is specific to this tool. |
Awesome, that makes it a lot easier. |
Closing based on your PR (thanks again!) |
Before I spent time cleaning up our fork and creating a PR for this, I wanted to check if this is something you would actually be interested in taking back at all.
Basically, I just wanted to use zstd compression instead (or in parallel?) with the current gzip compression. The primary reason is a much lower cache size and overall binary size.
There is also a slight (~20ms on my machine) improvement on decompression times, but it's probably not going to be noticeable in practice.
The text was updated successfully, but these errors were encountered: