Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Replace Zip and Flate packages. #205

Closed
wants to merge 1 commit into from

Conversation

calavera
Copy link

Use the optimized versions from github.com/klauspost/compress.
In smoke tests, compressing 7K files took 3 seconds, rather than 12
seconds with the official packages.

Use the optimized versions from github.com/klauspost/compress.
In smoke tests, compressing 7K files took 3 seconds, rather than 12
seconds with the official packages.

Signed-off-by: David Calavera <david.calavera@gmail.com>
@coolaj86
Copy link
Collaborator

@calavera Is this still relevant? Are the official packages still slow?

Also, what's the use case where this is a problem?

I feel like arc feels a niche of convenience, and we really value being light on dependencies, so I'm not sure if we want to bring those in or not. I'm open to it, but cautious.

Can we open this back up for discussion?

@calavera calavera closed this Sep 29, 2020
@calavera calavera deleted the compress_zip branch September 29, 2020 20:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants