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.

Sign up for GitHub

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

Jump to bottom

Support zstd compression in HTTP client and server #549

Closed
apolukhin opened this issue Apr 27, 2024 · 1 comment
Closed

Support zstd compression in HTTP client and server #549

apolukhin opened this issue Apr 27, 2024 · 1 comment
Labels
big A big feature help wanted We would appreciate PR

Comments

@apolukhin
Copy link
Member

ZSTD is faster to decompress and may have better compression than brotli https://t.me/experimentalchill/207

We should support the https://datatracker.ietf.org/doc/html/rfc8878#name-content-encoding

Internal ticket: 6039
@apolukhin apolukhin added help wanted We would appreciate PR big A big feature labels Apr 27, 2024
@IoplachkinI IoplachkinI mentioned this issue May 19, 2024
Closed
robot-piglet pushed a commit that referenced this issue May 29, 2024
@VS-CDR @apolukhin
feat server: add ZSTD decompression support
2f23f80 
Resolves issue #549

Benchmark results:
```
---------------------------------------------------------------
Benchmark                     Time             CPU   Iterations
---------------------------------------------------------------
ZstdDecompress/1024        4321 ns         4329 ns       161878
ZstdDecompress/2048        5288 ns         5293 ns       128786
ZstdDecompress/4096        7116 ns         7122 ns        97136
ZstdDecompress/8192       11499 ns        11502 ns        59946
ZstdDecompress/16384      16596 ns        16595 ns        42723
ZstdDecompress/32768      26929 ns        26926 ns        26055
GzipDecompress/1024       10459 ns        10457 ns        66482
GzipDecompress/2048       16552 ns        16550 ns        42502
GzipDecompress/4096       29337 ns        29338 ns        23916
GzipDecompress/8192       57317 ns        57314 ns        12272
GzipDecompress/16384     113640 ns       113633 ns         6156
GzipDecompress/32768     230311 ns       230269 ns         2952
```

Tests: протестировано CI
Pull Request resolved: #595

Co-authored-by: Yobama-png <i.oplachkin@gmail.com>
e82204f930195f8467c6fb9d83533872e9b75f38
@apolukhin
Copy link
Member Author

Resolved in 2f23f80

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
big A big feature help wanted We would appreciate PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@apolukhin