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

Add a feature to require minimum compression level #5

Open
cowtowncoder opened this issue Jun 5, 2013 · 0 comments
Open

Add a feature to require minimum compression level #5

cowtowncoder opened this issue Jun 5, 2013 · 0 comments

Comments

@cowtowncoder
Copy link
Owner

Currently any amount of compression, no matter how small, will result in compression being used. But due to cost of uncompression, it may not make sense to store compressed content for negligible amounts.
So: it would make sense to add a configuration value to specify maximum rate of content after compression (default could be something like 0.95 to require minimum of 5% compression) that will still be stored as compresed.

Note that checking may be inexact, due to limitations wrt streaming operations: that is, it is acceptable to use heuristics to estimate total compression, for example by compressing first N bytes of content.

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

No branches or pull requests

1 participant