codenotary / immudb / 25497188798

85%

Build Type

push

gh-ci

Committed by vchaindz

Commit Message

perf(sql): add reduceBool fast path to skip *Bool boxing in COUNT(*)

CmpBoolExp.reduce, BinBoolExp.reduce, and NotBoolExp.reduce each return
&Bool{...} per call. On the issue #2093 shape (4 comparisons + 3 ANDs in
WHERE), that is 7 boxed-Bool allocations per matched row.

Add a parallel reduceBool method to Bool, NullValue, CmpBoolExp, BinBoolExp,
and NotBoolExp that returns (val, isNull, err) directly. Composition through
AND/OR trees is handled by reduceBoolValueExp, which dispatches to the fast
path for known boolean exps and falls back to reduce + type-check for
unknown ones (LikeBoolExp, ExistsBoolExp, InListExp, ColSelector...). Slow
path semantics are preserved exactly: the fallback path is the same code
that previously lived inline in CountAllWithKeyFilter.

CountAllWithKeyFilter now calls reduceBoolValueExp instead of cond.reduce +
type assertion. Other reduce call sites are unchanged.

Coverage Stats

30 of 82 new or added lines in 2 files covered. (36.59%)

15 existing lines in 1 file now uncovered.

45204 of 53211 relevant lines covered (84.95%)

126091.27 hits per line