Skip to content

Commit e91b4d7

Browse files
committed
fix(clickhouse): drop a comment that broke webhook_deliveries_v1 migration parsing
The 034 migration had an inline comment containing a semicolon. The test ClickHouse migration runner splits statements on semicolons, so it cut the CREATE TABLE mid-statement and tried to parse the comment tail as SQL, which failed ClickHouse setup for every test suite that starts a container. Removing the comment leaves the CREATE as a single statement.
1 parent a15a2f9 commit e91b4d7

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

internal-packages/clickhouse/schema/034_create_webhook_deliveries_v1.sql

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,5 @@ CREATE TABLE IF NOT EXISTS trigger_dev.webhook_deliveries_v1
2727
ENGINE = ReplacingMergeTree(_version, _is_deleted)
2828
PARTITION BY toYYYYMM(created_at)
2929
ORDER BY (organization_id, project_id, environment_id, created_at, delivery_id)
30-
-- 60d retention aligned with the Postgres partition retention. ttl_only_drop_parts makes expiry a
31-
-- whole-part drop (monthly partitions); materialize_ttl_recalculate_only keeps TTL lightweight.
3230
TTL toDateTime(created_at) + INTERVAL 60 DAY
3331
SETTINGS ttl_only_drop_parts = 1, materialize_ttl_recalculate_only = 1;

0 commit comments

Comments
 (0)