-
Notifications
You must be signed in to change notification settings - Fork 22
Open
Description
Problem
The altimate-core tools don't expose a dialect parameter — they always target Snowflake internally. The sql_quality telemetry event supports an optional dialect field, but only the sql-* tools (analyze, optimize, schema-diff) populate it since they accept dialect as a user parameter.
Tools missing dialect support
altimate-core-validate.tsaltimate-core-fix.tsaltimate-core-correct.tsaltimate-core-semantics.tsaltimate-core-equivalence.tsaltimate-core-policy.tsaltimate-core-check.tsimpact-analysis.ts
Tools that already support dialect
sql-analyze.ts—dialectparameter, passed to dispatcher and telemetrysql-optimize.ts— sameschema-diff.ts— same
Proposed fix
- Add an optional
dialectparameter to all altimate-core tools (defaulting to"snowflake") - Pass the dialect through to the Dispatcher call
- Include
dialectin metadata so telemetry picks it up
This enables per-dialect telemetry segmentation and prepares these tools for multi-dialect support (postgres, bigquery, duckdb, etc.).
Context
Identified during PR #446 (sql_quality telemetry). Rather than hardcoding dialect: "snowflake", we made the telemetry field optional — tools that don't set it simply omit it from the event.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels