Skip to content

fix: bound profiler memory growth by trimming interned call stacks - #5503

Merged
jamescrosswell merged 20 commits into
mainfrom
fix/5469-trim-live-session-state
Aug 31, 2026
Merged

jamescrosswell merged 20 commits into
mainfrom
fix/5469-trim-live-session-state

Make the call stack budget per-factory rather than a static

908f26f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 31, 2026 in 0s

74.67% (-0.07%) compared to 4e0e0de

View this Pull Request on Codecov

74.67% (-0.07%) compared to 4e0e0de

Details

Codecov Report

❌ Patch coverage is 7.69231% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.67%. Comparing base (4e0e0de) to head (908f26f).
⚠️ Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...ry.Profiling/SamplingTransactionProfilerFactory.cs 14.28% 12 Missing ⚠️
src/Sentry.Profiling/SampleProfileBuilder.cs 0.00% 6 Missing and 1 partial ⚠️
src/Sentry.Profiling/SampleProfilerSession.cs 0.00% 4 Missing ⚠️
...rc/Sentry.Profiling/SamplingTransactionProfiler.cs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5503      +/-   ##
==========================================
- Coverage   74.74%   74.67%   -0.07%     
==========================================
  Files         513      515       +2     
  Lines       18829    18909      +80     
  Branches     3682     3691       +9     
==========================================
+ Hits        14074    14121      +47     
- Misses       3875     3904      +29     
- Partials      880      884       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.