From 06185227d1c9b1800f15433b24d7470b1a1da56e Mon Sep 17 00:00:00 2001 From: getsentry-bot Date: Thu, 20 Jun 2024 20:40:37 +0000 Subject: [PATCH] Revert "fix(group-attributes): Modify the column type for `group_first_release_id` to bigint (#6040)" This reverts commit 270e64b5bdf9d39fabdfa762c113603982c18cab. Co-authored-by: snigdhas <16563948+snigdhas@users.noreply.github.com> --- snuba/migrations/group_loader.py | 1 - .../0004_fix_group_first_release_id_type.py | 47 ------------------- 2 files changed, 48 deletions(-) delete mode 100644 snuba/snuba_migrations/group_attributes/0004_fix_group_first_release_id_type.py diff --git a/snuba/migrations/group_loader.py b/snuba/migrations/group_loader.py index 3f35ec04b9..2cbc098784 100644 --- a/snuba/migrations/group_loader.py +++ b/snuba/migrations/group_loader.py @@ -400,7 +400,6 @@ def get_migrations(self) -> Sequence[str]: "0001_group_attributes", "0002_add_priority_to_group_attributes", "0003_add_first_release_id_to_group_attributes", - "0004_fix_group_first_release_id_type", ] diff --git a/snuba/snuba_migrations/group_attributes/0004_fix_group_first_release_id_type.py b/snuba/snuba_migrations/group_attributes/0004_fix_group_first_release_id_type.py deleted file mode 100644 index cd441c6c25..0000000000 --- a/snuba/snuba_migrations/group_attributes/0004_fix_group_first_release_id_type.py +++ /dev/null @@ -1,47 +0,0 @@ -from typing import Sequence - -from snuba.clickhouse.columns import UUID, Column, UInt -from snuba.clusters.storage_sets import StorageSetKey -from snuba.migrations import migration, operations -from snuba.migrations.columns import MigrationModifiers as Modifiers -from snuba.migrations.operations import OperationTarget, SqlOperation - - -class Migration(migration.ClickhouseNodeMigration): - blocking = False - - def forwards_ops(self) -> Sequence[SqlOperation]: - return [ - operations.ModifyColumn( - storage_set=StorageSetKey.GROUP_ATTRIBUTES, - table_name="group_attributes_local", - column=Column( - "group_first_release_id", UInt(64, Modifiers(nullable=True)) - ), - target=OperationTarget.LOCAL, - ), - operations.ModifyColumn( - storage_set=StorageSetKey.GROUP_ATTRIBUTES, - table_name="group_attributes_dist", - column=Column( - "group_first_release_id", UInt(64, Modifiers(nullable=True)) - ), - target=OperationTarget.DISTRIBUTED, - ), - ] - - def backwards_ops(self) -> Sequence[SqlOperation]: - return [ - operations.ModifyColumn( - storage_set=StorageSetKey.GROUP_ATTRIBUTES, - table_name="group_attributes_dist", - column=Column("group_first_release_id", UUID(Modifiers(nullable=True))), - target=OperationTarget.DISTRIBUTED, - ), - operations.ModifyColumn( - storage_set=StorageSetKey.GROUP_ATTRIBUTES, - table_name="group_attributes_local", - column=Column("group_first_release_id", UUID(Modifiers(nullable=True))), - target=OperationTarget.LOCAL, - ), - ]