Skip to content

fix(router): resolve plan cache memory leaks on config reload#3023

Closed
dkorittki wants to merge 6 commits into
mainfrom
dominik/eng-9822-fix-plancache-related-leaks
Closed

fix(router): resolve plan cache memory leaks on config reload#3023
dkorittki wants to merge 6 commits into
mainfrom
dominik/eng-9822-fix-plancache-related-leaks

fix: do not wait for pending writes on plancache on mux shutdown

b4f4018
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 26, 2026 in 1s

61.42% (target 30.00%)

View this Pull Request on Codecov

61.42% (target 30.00%)

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.42%. Comparing base (f144989) to head (b4f4018).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3023       +/-   ##
===========================================
+ Coverage   46.53%   61.42%   +14.88%     
===========================================
  Files        1120      259      -861     
  Lines      154966    30055   -124911     
  Branches    10060        0    -10060     
===========================================
- Hits        72107    18460    -53647     
+ Misses      81025    10107    -70918     
+ Partials     1834     1488      -346     
Files with missing lines Coverage Δ
router/core/graph_server.go 85.70% <100.00%> (+0.04%) ⬆️
router/core/operation_planner.go 71.11% <ø> (-0.32%) ⬇️
router/pkg/slowplancache/slow_plan_cache.go 99.05% <100.00%> (+0.05%) ⬆️

... and 863 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.