Skip to content

Conversation

@alexander-alderman-webb
Copy link
Contributor

Description

Introduce a hard limit on the number of elements in the log batcher. The size constraint prevents the log batcher from using too much memory.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner November 5, 2025 08:50
@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.97%. Comparing base (faa327c) to head (83e206a).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@ Coverage Diff @@ ## master #5069 +/- ## ======================================= Coverage 83.96% 83.97% ======================================= Files 179 179 Lines 17943 17948 +5 Branches 3192 3193 +1 ======================================= + Hits 15066 15071 +5  Misses 1905 1905 Partials 972 972 
Files with missing lines Coverage Δ
sentry_sdk/_log_batcher.py 80.68% <100.00%> (+1.16%) ⬆️
sentry_sdk/client.py 83.80% <100.00%> (ø)
@alexander-alderman-webb alexander-alderman-webb merged commit 26391d6 into master Nov 6, 2025
175 of 185 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/log-batcher-upper-bound branch November 6, 2025 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants