From 004b5f9a28314a07f860ca8398857dc2f2ff8364 Mon Sep 17 00:00:00 2001 From: Daniel Song Date: Mon, 30 Mar 2026 02:05:31 -0700 Subject: [PATCH 1/3] docs: add tiered database support matrix to README Replace the flat "Supported" list with a tiered matrix showing Production, Stable, Limited, and Experimental tiers with known limitations per database. Users can now self-select appropriate databases before investing setup time. Closes #32 Co-Authored-By: Claude Opus 4.6 --- README.md | 36 +++++++++++++++++++++--------------- 1 file changed, 21 insertions(+), 15 deletions(-) diff --git a/README.md b/README.md index 076fa0fa..650bb5e0 100644 --- a/README.md +++ b/README.md @@ -48,21 +48,27 @@ for sign, row in diff: ## Supported Databases -| Database | Status | -|-------------|--------| -| PostgreSQL | Supported | -| MySQL | Supported | -| Snowflake | Supported | -| BigQuery | Supported | -| Databricks | Supported | -| Redshift | Supported | -| DuckDB | Supported | -| Presto | Supported | -| Trino | Supported | -| Oracle | Supported | -| MS SQL | Supported | -| ClickHouse | Supported | -| Vertica | Supported | +| Database | Tier | Known Limitations | +|----------|------|-------------------| +| PostgreSQL | Production | — | +| MySQL | Production | — | +| DuckDB | Production | — | +| Redshift | Stable | Extends PostgreSQL driver | +| Snowflake | Stable | No optimizer hints | +| Presto | Stable | — | +| Vertica | Stable | — | +| Databricks | Stable | No unique constraint support | +| MsSQL | Limited | No session timezone; OFFSET requires ORDER BY | +| BigQuery | Limited | No session timezone; array/struct compared as JSON | +| ClickHouse | Limited | No session timezone; complex decimal normalization | +| Oracle | Limited | No OFFSET support; no EXPLAIN | +| Trino | Experimental | Minimal driver extending Presto; may have SQL divergences | + +**Tier definitions:** +- **Production** — All methods implemented, dedicated tests, CI coverage +- **Stable** — Core functionality works, minor limitations noted above +- **Limited** — Usable but missing some features; cross-database comparisons may have edge cases +- **Experimental** — Minimal implementation, use with caution ## dbt Integration From 9d4afe52046ea479e882a71488cf90809b7eb792 Mon Sep 17 00:00:00 2001 From: Daniel Song Date: Mon, 30 Mar 2026 02:12:31 -0700 Subject: [PATCH 2/3] fix: correct MsSQL and Presto limitation descriptions in support matrix - MsSQL: change "OFFSET requires ORDER BY" to "no OFFSET support" (the driver raises NotImplementedError for non-zero offset) - Presto: add "No primary key detection; fixed precision defaults" (lacks SUPPORTS_PRIMARY_KEY; hardcodes datetime/numeric precision to 3) Co-Authored-By: Claude Opus 4.6 --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 650bb5e0..17cc21cb 100644 --- a/README.md +++ b/README.md @@ -55,10 +55,10 @@ for sign, row in diff: | DuckDB | Production | — | | Redshift | Stable | Extends PostgreSQL driver | | Snowflake | Stable | No optimizer hints | -| Presto | Stable | — | +| Presto | Stable | No primary key detection; fixed precision defaults | | Vertica | Stable | — | | Databricks | Stable | No unique constraint support | -| MsSQL | Limited | No session timezone; OFFSET requires ORDER BY | +| MsSQL | Limited | No session timezone; no OFFSET support | | BigQuery | Limited | No session timezone; array/struct compared as JSON | | ClickHouse | Limited | No session timezone; complex decimal normalization | | Oracle | Limited | No OFFSET support; no EXPLAIN | From b8cd9c66cac69cb51c229238247fedec52856242 Mon Sep 17 00:00:00 2001 From: Daniel Song Date: Mon, 30 Mar 2026 02:18:38 -0700 Subject: [PATCH 3/3] fix: remove stale ClickHouse timezone limitation from support matrix ClickHouse set_timezone_to_utc() is now implemented (PR #52), so the "No session timezone" limitation no longer applies. Co-Authored-By: Claude Opus 4.6 --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 17cc21cb..14ccaa15 100644 --- a/README.md +++ b/README.md @@ -60,7 +60,7 @@ for sign, row in diff: | Databricks | Stable | No unique constraint support | | MsSQL | Limited | No session timezone; no OFFSET support | | BigQuery | Limited | No session timezone; array/struct compared as JSON | -| ClickHouse | Limited | No session timezone; complex decimal normalization | +| ClickHouse | Limited | Complex decimal normalization | | Oracle | Limited | No OFFSET support; no EXPLAIN | | Trino | Experimental | Minimal driver extending Presto; may have SQL divergences |