Skip to content

feat: add native SQLite aggregation - #187

Merged
trakhimenok merged 5 commits into
mainfrom
dtql-group-by-aggregation
Sep 20, 2026
Merged

trakhimenok merged 5 commits into
mainfrom
dtql-group-by-aggregation

Conversation

@trakhimenok

Copy link
Copy Markdown
Contributor

Summary

  • compile DALgo aggregation queries into native SQLite GROUP BY, aggregate, HAVING, ordering, offset, and limit clauses
  • preserve DALgo numeric, collation, placeholder, and non-finite result semantics
  • add native, streaming, and hash parity coverage

This is stacked on PR #186 for negative projection and consumes the exact DALgo aggregation commit from dal-go/dalgo#174.

Verification

  • GOWORK=off go test ./...
  • GOWORK=off go vet ./...
  • git diff --check
  • cross-repository parity and adversarial review completed

WB effort: dtql-group-by-aggregation

Base automatically changed from negative-column-projection to main September 20, 2026 20:21
@trakhimenok
trakhimenok marked this pull request as ready for review September 20, 2026 20:28
@trakhimenok

Copy link
Copy Markdown
Contributor Author

Review by unknown@codex@unknown (head ad7dc52b5113)
Reviewed-Head: ad7dc52

Reviewed the exact reconciled head after merging current main. The resolution retains wildcard projection and identity behavior together with aggregation columns, aliases, finite-result validation, and the newer DALgo aggregation pin. Standalone tests, vet, golangci-lint, diff checks, and push hooks pass.

@trakhimenok
trakhimenok enabled auto-merge (squash) September 20, 2026 20:33
@trakhimenok
trakhimenok merged commit 91d5bf1 into main Sep 20, 2026
5 checks passed
@trakhimenok
trakhimenok deleted the dtql-group-by-aggregation branch September 20, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant