Skip to content
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

Allow caching without compression #1535

Open
SanjayVas opened this issue Sep 21, 2023 · 4 comments · May be fixed by #1877
Open

Allow caching without compression #1535

SanjayVas opened this issue Sep 21, 2023 · 4 comments · May be fixed by #1877
Labels
enhancement New feature or request

Comments

@SanjayVas
Copy link

Allowing one to choose not to compress the tar archive when caching. This allows for the useful tradeoff between time and cache entry size. Note that cache size is already bounded by the total per-repo limit.

Compression frequently takes up the majority of the time spent in a cache save step, so this could offer a significant time savings at the cost of size.

This is a more specific version of #544.

@SanjayVas SanjayVas added the enhancement New feature or request label Sep 21, 2023
@planetmarshall
Copy link

This would also be useful for users of the conan package manager which has its own method of creating cache archives and the purpose of the cache action would purely be to upload and download the prepared archives.

@anton-danielsson
Copy link

This would also be useful for users of the conan package manager which has its own method of creating cache archives and the purpose of the cache action would purely be to upload and download the prepared archives.

Exactly what brought me here 🙂
Would be very nice to have this option.

@andrewchambers
Copy link

Compressing/decompressing already compressed data is a huge waste of resources, avoiding that makes a lot of sense.

@planetmarshall
Copy link

Would you accept a PR for this feature? Would probably also implement #544

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants