pyroscope icon indicating copy to clipboard operation
pyroscope copied to clipboard

fix(v2): make segment upload hedging limiter respect context

Open kolesnikovae opened this issue 6 months ago • 1 comments

Previously, if the segment writer failed to upload to object storage, it could hit the hedge request rate limit and block clients ignoring the upload timeout context. This could lead to unbounded memory usage and potential OOM if object storage became unavailable. This change ensures the limiter respects the upload context.

kolesnikovae avatar Jun 13 '25 12:06 kolesnikovae

cc @alsoba13

kolesnikovae avatar Jun 13 '25 12:06 kolesnikovae

Sorry for the delay.

LGTM

alsoba13 avatar Jun 20 '25 06:06 alsoba13