Commit 29acf7b
authored
Match the version being adopted in altimate-dags PR #1024 so all
downstream consumers (`altimate-dags`, airflow workers via
`altimate-datapilot-cli`) resolve to the same C-accelerated sqlglot
release. The 6.8x parsing speedup landing in altimate-dags benefits
any DAG path that reaches into datapilot-cli's `SqlCheck`.
`SqlCheck` (`src/datapilot/core/platforms/dbt/insights/sql/sql_check.py`)
is the only place that imports `sqlglot`. Its optimizer-rule pipeline
(`pushdown_projections`, `normalize`, `unnest_subqueries`,
`eliminate_subqueries`, `eliminate_joins`, `eliminate_ctes`) inspects
each rule via `inspect.getfullargspec(rule).args`. Verified that
sqlglot[c] 30.7.0 still exposes proper signatures via mypyc — no
compat shim required.
Non-breaking validation:
- Existing test suite: **77/77 pass** on both 25.30.0 (old pin) and
30.7.0 (new pin).
- SqlCheck output is byte-identical between 25.30.0 and 30.7.0 on a
7-query fixture covering filter pushdown, CTE chains, unused joins,
IN-subqueries, DISTINCT dedup, OR/AND filters, and a no-op query.
Same 6 insights, same recommended optimized SQL, same rule names.
Refs: AltimateAI/altimate-dags#1024
1 parent 7c9043e commit 29acf7b
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | | - | |
| 70 | + | |
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
| |||
0 commit comments