gluesql / gluesql / 24926141446

99%

Build Type

push

github

Committed by web-flow

Commit Message

Bind aggregate slots during planning (#1895)

Move aggregate slot assignment out of runtime execution and into a dedicated planning pass.
Select statements now carry planner-assigned aggregate slots,
and aggregate expressions reference those slots during evaluation
instead of relying on execution-time aggregate lookup.

Update aggregate execution state to use slot-indexed values, including empty global aggregate handling,
and require planned aggregate bindings at execution time.
Apply aggregate planning across query-bearing statements and storage planner implementations.

Also tighten CTAS behavior around planned select output, avoid copying source schemas for joined CTAS queries,
and add regression coverage for CTAS aggregate results,
empty COUNT(*) metadata behavior, aggregate subqueries in DML planning,
and AST equality/hash semantics for planner-only aggregate fields.

Coverage Stats

902 of 953 new or added lines in 33 files covered. (94.65%)

1 existing line in 1 file now uncovered.

43270 of 44099 relevant lines covered (98.12%)

72532.61 hits per line