Skip to content

perf: sort window rows in place#376

Merged
KKould merged 1 commit into
mainfrom
perf/window-inline-sort
Jul 13, 2026
Merged

perf: sort window rows in place#376
KKould merged 1 commit into
mainfrom
perf/window-inline-sort

perf: sort window rows in place

52bda81
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 13, 2026 in 1s

92.85% (+0.00%) compared to c19209e

View this Pull Request on Codecov

92.85% (+0.00%) compared to c19209e

Details

Codecov Report

❌ Patch coverage is 98.11321% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.85%. Comparing base (c19209e) to head (52bda81).

Files with missing lines Patch % Lines
src/execution/dql/sort.rs 96.29% 2 Missing ⚠️
src/execution/dql/window.rs 97.56% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #376   +/-   ##
=======================================
  Coverage   92.84%   92.85%           
=======================================
  Files         249      249           
  Lines       41790    41844   +54     
=======================================
+ Hits        38802    38856   +54     
  Misses       2988     2988           
Files with missing lines Coverage Δ
src/binder/window.rs 95.89% <100.00%> (-0.22%) ⬇️
src/execution/dql/window/function.rs 97.22% <100.00%> (ø)
src/optimizer/heuristic/optimizer.rs 97.09% <100.00%> (ø)
src/optimizer/rule/implementation/dql/window.rs 100.00% <100.00%> (ø)
src/planner/operator/mod.rs 95.81% <100.00%> (ø)
src/planner/operator/visitor.rs 98.93% <100.00%> (+<0.01%) ⬆️
src/planner/operator/visitor_mut.rs 99.66% <100.00%> (ø)
src/planner/operator/window.rs 89.47% <100.00%> (+4.85%) ⬆️
src/execution/dql/window.rs 98.98% <97.56%> (-1.02%) ⬇️
src/execution/dql/sort.rs 87.83% <96.29%> (+0.05%) ⬆️

... and 1 file with indirect coverage changes

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