dal-go / dalgo2sql / 25912606450

89%

push

github

feat(dalgo2sql): emit ANSI/SQLite-compatible LIMIT N instead of T-SQL TOP N

Adds sqlite_emit.go with an emitSQL(q) shim that rewrites dalgo's
"SELECT TOP N" prefix into a trailing "LIMIT N" clause. reader_base.go
calls emitSQL() instead of q.String() directly when handling
StructuredQuery, so SQLite (and PostgreSQL/MySQL) accept the SQL
without local rewrites in callers.

Stopgap until upstream dalgo gains dialect-aware emission — tracked
under the `dalgo-dialect-aware-sql-emission` Idea in
dal-go/dalgo/spec/ideas/. The shim disappears once that lands.

Triggered by datatug-cli's db-copy filtering Feature (Task 7
--limit support against SQLite Chinook fixture).

0 of 14 new or added lines in 2 files covered. (0.0%)

871 of 1169 relevant lines covered (74.51%)

0.86 hits per line