-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add --compress=best
#22
Comments
I was curious,
It's not really clear to me that even The biggest wins for zstd are when a pre-trained dictionary is provided, but that makes everything stateful and hence harms reproducibility. (zstd vs gzip on my test case has zstd being ~10x faster to compress at default levels than |
Hmm, I'm not sure I agree. Releases are done rarely, their artifacts live forever, and 3 more seconds (or 30) isn't that costly. (gzip's relative performance isn't directly relevant here, since the reason to use gzip is compatibility rather than performance.) This is mitigated by the fact that vendor tarballs are probably not used very often, so the worldwide storage/transfer cost is limited. |
I have absolutely no objections to adding I'd admit my perspective on this is a bit tainted by the fact that right now at this moment I am running this program frequently for local interactive testing, so speed is better than compression 😄 In the end though for storage, the worldwide growth in high-definition video, big data in general, etc. has driven immense gains in storage technologies, things like SMR hard drives. My kids just yesterday recorded multiple 5-10 minute videos of themselves singing and I am sure they have no idea how much storage that took on my phone. The vendor tarballs are tiny peanuts compared to this stuff. |
The
tar
compression flags invokegzip
andzstd
with their default compression levels, which are not especially tight. Since we're creating long-term archives, we should probably compress as tightly as reasonably possible. That does require that we invoke the compressor separately.The text was updated successfully, but these errors were encountered: