connect: Use klauspost/compress/gzip decompression

grafana / pyroscope

Continuous Profiling Platform. Debug performance issues down to a single line of code

https://grafana.com/oss/pyroscope/

GNU Affero General Public License v3.0

9.65k stars 576 forks source link

connect: Use klauspost/compress/gzip decompression #3314

Closed simonswine closed 1 month ago

simonswine commented 1 month ago

Now that we have shared options between all connect Handlers/Clients it is easy to swap out compress/gzip with klauspost/compress/gzip.

At the same time this makes sure that all handlers and clients use the same compressor pool. Otherwise every new handler and client maintains its own pool (And we create a lot of clients dynamically).

Relies on the work of #3310

simonswine commented 1 month ago

Biggest effect on the distibutor is in the clients sharing the compression pools. Saves about 5% of allocations