grafana / grafana/pyroscope

Rate limiting on compressed data

Open
#2,411 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
11.7k
Forks
802
Avg merge
1d 19h
Merged PRs (30d)
80

Description

Right now rate limiting is done on uncompressed data which forces us to spend some CPU cycles even if we reject the payload.

We have a consensus that we should most likely do this on compressed data instead and reject it as soon as possible.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the rate-limiting path and the compressed-payload handling in the Pyroscope codebase. Trace where payloads are currently decompressed relative to rejection, then identify the existing tests covering rate limiting or compressed data. Done means rejected payloads are stopped using compressed data before unnecessary decompression, with relevant behavior covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend, performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.