From 217f514657bcf008a9973b9fbffb6cd414b5d98d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabr=C3=ADzio=20de=20Royes=20Mello?= Date: Wed, 21 Sep 2022 10:15:32 -0300 Subject: [PATCH] Fix continuous aggregate migrate check constraint Instances upgraded to 2.8.0 will end up with a wrong check constraint in catalog table `continuous_aggregate_migrate_plan_step`. Fixed it by removing and adding the constraint with the correct checks. Fix #4727 --- CHANGELOG.md | 2 ++ sql/updates/latest-dev.sql | 8 ++++++++ sql/updates/reverse-dev.sql | 8 ++++++++ 3 files changed, 18 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 5dfc84529..a67c1650a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -24,10 +24,12 @@ argument or resolve the type ambiguity by casting to the intended type. * #4685 Improve chunk exclusion for space dimensions * #4696 Report warning when enabling compression on hypertable * #4676 Fix a deadlock when decompressing chunks and performing SELECTs +* #4739 Fix continuous aggregate migrate check constraint **Thanks** * @maxtwardowski for reporting problems with chunk exclusion and space dimensions * @yuezhihan for reporting GROUP BY error when setting compress_segmentby with an enum column +* @carobme for reporting constraint error during continuous aggregate migration ## 2.8.0 (2022-08-30) diff --git a/sql/updates/latest-dev.sql b/sql/updates/latest-dev.sql index da24bd8c0..51e29e4d0 100644 --- a/sql/updates/latest-dev.sql +++ b/sql/updates/latest-dev.sql @@ -46,3 +46,11 @@ BEGIN WHERE mat_hypertable_id OPERATOR(pg_catalog.=) _cagg_data.mat_hypertable_id; END; $BODY$; + +-- Issue #4727 +ALTER TABLE _timescaledb_catalog.continuous_agg_migrate_plan_step + DROP CONSTRAINT IF EXISTS continuous_agg_migrate_plan_step_check2; + +ALTER TABLE _timescaledb_catalog.continuous_agg_migrate_plan_step + ADD CONSTRAINT continuous_agg_migrate_plan_step_check2 + CHECK (type IN ('CREATE NEW CAGG', 'DISABLE POLICIES', 'COPY POLICIES', 'ENABLE POLICIES', 'SAVE WATERMARK', 'REFRESH NEW CAGG', 'COPY DATA', 'OVERRIDE CAGG', 'DROP OLD CAGG')); diff --git a/sql/updates/reverse-dev.sql b/sql/updates/reverse-dev.sql index 884c51978..783b5158b 100644 --- a/sql/updates/reverse-dev.sql +++ b/sql/updates/reverse-dev.sql @@ -45,3 +45,11 @@ BEGIN WHERE mat_hypertable_id OPERATOR(pg_catalog.=) _cagg_data.mat_hypertable_id; END; $BODY$; + +-- Issue #4727 +ALTER TABLE _timescaledb_catalog.continuous_agg_migrate_plan_step + DROP CONSTRAINT IF EXISTS continuous_agg_migrate_plan_step_check2; + +ALTER TABLE _timescaledb_catalog.continuous_agg_migrate_plan_step + ADD CONSTRAINT continuous_agg_migrate_plan_step_check2 + CHECK (type IN ('CREATE NEW CAGG', 'DISABLE POLICIES', 'COPY POLICIES', 'ENABLE POLICIES', 'SAVE WATERMARK', 'REFRESH NEW CAGG', 'COPY DATA'));