Fix the migration

matter-labs · Sep 6, 2024 · 0c4e753 · 0c4e753
1 parent 42db941
commit 0c4e753
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/core/lib/dal/migrations/20240905123059_rename-recurision-scheduler-level-vk-hash.down.sql b/core/lib/dal/migrations/20240905123059_rename-recurision-scheduler-level-vk-hash.down.sql
@@ -1,2 +1,3 @@
 UPDATE protocol_patches SET recursion_scheduler_level_vk_hash = snark_wrapper_vk_hash WHERE recursion_scheduler_level_vk_hash = ''::bytea;
 ALTER TABLE protocol_patches DROP COLUMN snark_wrapper_vk_hash;
+ALTER TABLE protocol_patches ALTER COLUMN recursion_scheduler_level_vk_hash DROP DEFAULT;
diff --git a/core/lib/dal/migrations/20240905123059_rename-recurision-scheduler-level-vk-hash.up.sql b/core/lib/dal/migrations/20240905123059_rename-recurision-scheduler-level-vk-hash.up.sql
@@ -1,6 +1,8 @@
-ALTER TABLE protocol_patches ADD COLUMN snark_wrapper_vk_hash BYTEA NOT NULL;
+ALTER TABLE protocol_patches ADD COLUMN snark_wrapper_vk_hash BYTEA NOT NULL DEFAULT ''::bytea;
 ALTER TABLE protocol_patches ALTER COLUMN recursion_scheduler_level_vk_hash SET DEFAULT ''::bytea;
 UPDATE protocol_patches SET snark_wrapper_vk_hash = recursion_scheduler_level_vk_hash;
+-- Default was only needed to migrate old rows, we don't want this field to be forgotten by accident after migration.
+ALTER TABLE protocol_patches ALTER COLUMN snark_wrapper_vk_hash DROP DEFAULT;
 
 -- Old column should be removed once the migration is on the mainnet.
 COMMENT ON COLUMN protocol_patches.recursion_scheduler_level_vk_hash IS 'This column is deprecated and will be removed in the future. Use snark_wrapper_vk_hash instead.';
diff --git a/...b/prover_dal/migrations/20240905124208_rename-recurision-scheduler-level-vk-hash.down.sql b/...b/prover_dal/migrations/20240905124208_rename-recurision-scheduler-level-vk-hash.down.sql
@@ -1,2 +1,3 @@
 UPDATE prover_fri_protocol_versions SET recursion_scheduler_level_vk_hash = snark_wrapper_vk_hash WHERE recursion_scheduler_level_vk_hash = ''::bytea;
 ALTER TABLE prover_fri_protocol_versions DROP COLUMN snark_wrapper_vk_hash;
+ALTER TABLE prover_fri_protocol_versions ALTER COLUMN recursion_scheduler_level_vk_hash DROP DEFAULT;
diff --git a/...lib/prover_dal/migrations/20240905124208_rename-recurision-scheduler-level-vk-hash.up.sql b/...lib/prover_dal/migrations/20240905124208_rename-recurision-scheduler-level-vk-hash.up.sql
@@ -1,6 +1,8 @@
-ALTER TABLE prover_fri_protocol_versions ADD COLUMN snark_wrapper_vk_hash BYTEA NOT NULL;
+ALTER TABLE prover_fri_protocol_versions ADD COLUMN snark_wrapper_vk_hash BYTEA NOT NULL DEFAULT ''::bytea;
 ALTER TABLE prover_fri_protocol_versions ALTER COLUMN recursion_scheduler_level_vk_hash SET DEFAULT ''::bytea;
 UPDATE prover_fri_protocol_versions SET snark_wrapper_vk_hash = recursion_scheduler_level_vk_hash;
+-- Default was only needed to migrate old rows, we don't want this field to be forgotten by accident after migration.
+ALTER TABLE prover_fri_protocol_versions ALTER COLUMN snark_wrapper_vk_hash DROP DEFAULT;
 
 -- Old column should be removed once the migration is on the mainnet.
 COMMENT ON COLUMN prover_fri_protocol_versions.recursion_scheduler_level_vk_hash IS 'This column is deprecated and will be removed in the future. Use snark_wrapper_vk_hash instead.';