Skip to content

make compressor configurable#9914

Closed
kkmuffme wants to merge 2 commits intovimeo:masterfrom
kkmuffme:compressor-configurable-lz4-deflate-off
Closed

make compressor configurable#9914
kkmuffme wants to merge 2 commits intovimeo:masterfrom
kkmuffme:compressor-configurable-lz4-deflate-off

Conversation

@kkmuffme
Copy link
Copy Markdown
Contributor

Make compressor configurable for lz4, deflate (gzip/zlib) or off.
Fix #9905

@kkmuffme kkmuffme force-pushed the compressor-configurable-lz4-deflate-off branch from 12e49f9 to ad3fac9 Compare June 13, 2023 15:03
which was removed in vimeo#9889 - now we won't report for every error but if there are at least 10 errors (chosen arbitrarily)
@kkmuffme kkmuffme force-pushed the compressor-configurable-lz4-deflate-off branch from 49fc00e to 90b047c Compare June 13, 2023 15:20
@kkmuffme kkmuffme marked this pull request as ready for review June 13, 2023 15:20
@kkmuffme
Copy link
Copy Markdown
Contributor Author

@orklah please merge

@kkmuffme
Copy link
Copy Markdown
Contributor Author

@orklah please merge.

If people really want to change the cache behavior from what we had previously to completely ignore failures, then that should be a separate PR, where the errors for cache writes are ignored too to keep it consistent.

kkmuffme added a commit to kkmuffme/psalm that referenced this pull request Jun 18, 2023
which was removed in vimeo#9889 - now we won't report for every error but if there are at least 10 errors (chosen arbitrarily)

Cherry picked from vimeo#9914
@kkmuffme kkmuffme closed this Jun 18, 2023
@kkmuffme kkmuffme deleted the compressor-configurable-lz4-deflate-off branch June 18, 2023 21:41
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

Successfully merging this pull request may close these issues.

GZIP - disable/set level because performance impact is bad

4 participants