stephenafamo / scan

90%

main: 91%

LAST BUILD ON BRANCH scan-plan-once

branch: scan-plan-once

CHANGE BRANCH

x

Reset

• scan-plan-once
• fix/close-rows-in-fromrows
• fix/each-rows-err-check
• main
• perf/reuse-row-scan-buffers
• perf/schedule-scan-by-index
• refs/tags/v0.1.0
• refs/tags/v0.1.1
• refs/tags/v0.2.0
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.5.0
• refs/tags/v0.6.0
• refs/tags/v0.6.1
• refs/tags/v0.6.2
• refs/tags/v0.7.0
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• revert-3-named-args-v2
• scan-rowskip

Build # 29675134524

Build Type

Pull #12

github

Committed by sandonemaki

Commit Message

perf: reuse the per-query link and destination buffers across rows

scanOneRow calls before/scan/after strictly in sequence for each row,
and the mapper factory runs once per query, so per-query closures can
carry buffers that are reused across rows — the same reasoning behind
the links slice in Mod and the scan-destination buffers in Row.

Reuse the allOptions destinations slice (and its boxed interface
value) across rows, and carry the regular path's row value in a small
reusable holder instead of boxing a new interface value on every row.

This removes two allocations per row from the allOptions path and one
from the regular path.

Pull Request Pull Request #12: perf: build the struct mapping plan once per query, not once per row

Coverage Stats

87 of 87 new or added lines in 2 files covered. (100.0%)

623 of 690 relevant lines covered (90.29%)

43.24 hits per line