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

gzip: change compression level to the default #2437

Conversation

alban
Copy link
Contributor

@alban alban commented Apr 12, 2017

We want the middle ground between a small compression size, a fast
compression time and a fast decompression time.

Tests suggest that the default compression level is better than the
maximum compression level: although the reports are 4% bigger and
decompress slower, they compress 33% faster.

See discussion on #1457 (comment)

We want the middle ground between a small compression size, a fast
compression time and a fast decompression time.

Tests suggest that the default compression level is better than the
maximum compression level: although the reports are 4% bigger and
decompress slower, they compress 33% faster.

See discussion on weaveworks#1457 (comment)
@alban
Copy link
Contributor Author

alban commented Apr 12, 2017

@bboreham what do you think of this change?

@alban
Copy link
Contributor Author

alban commented Apr 13, 2017

@ekimekim: PTAL

@ekimekim
Copy link
Contributor

Out of interest, did you test any other compression algorithms? For example, lz4 is generally my goto for fast compression/decompression, but I don't know how badly it would increase compressed size.

@ekimekim
Copy link
Contributor

(feel free to merge as-is without answering, it was just for my own curiosity)

@bboreham
Copy link
Collaborator

The problem with moving to a different compression scheme is that we'd have to change both client and server, and maintain backwards-compatibility in the server.

With gzip, the same server code will decode data encoded at any level of compression.

@alban
Copy link
Contributor Author

alban commented Apr 18, 2017

Out of interest, did you test any other compression algorithms? For example, lz4 is generally my goto for fast compression/decompression, but I don't know how badly it would increase compressed size.

I tested other compression algorithms but outside of Scope and I didn't try lz4:
#1457 (comment)

I used a msgpack generated by Scope though.

(feel free to merge as-is without answering, it was just for my own curiosity)

@ekimekim I don't have merge rights, so I'll let you do it.

@bboreham bboreham merged commit c944225 into weaveworks:master Apr 18, 2017
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.

3 participants