From cdfd3ba672ebae9000ea0c3a8fb4591c52d05d53 Mon Sep 17 00:00:00 2001 From: getsentry-bot Date: Thu, 18 Apr 2024 21:00:16 +0000 Subject: [PATCH] Revert "feat(replays): add replay_id uuid processors to config (#5791)" This reverts commit 8ede52abd1860f16d6eb0431618bb8e7fbf3fe2d. Co-authored-by: JoshFerge <1976777+JoshFerge@users.noreply.github.com> --- .../datasets/configuration/discover/storages/discover.yaml | 5 +++-- snuba/datasets/configuration/events/storages/errors.yaml | 6 +++++- .../datasets/configuration/events/storages/errors_ro.yaml | 7 ++++++- .../configuration/transactions/storages/transactions.yaml | 3 +-- 4 files changed, 15 insertions(+), 6 deletions(-) diff --git a/snuba/datasets/configuration/discover/storages/discover.yaml b/snuba/datasets/configuration/discover/storages/discover.yaml index 1c3b439d2e..ee42a5c869 100644 --- a/snuba/datasets/configuration/discover/storages/discover.yaml +++ b/snuba/datasets/configuration/discover/storages/discover.yaml @@ -88,7 +88,6 @@ schema: args: { schema_modifiers: [nullable], size: 64 }, }, { name: deleted, type: UInt, args: { size: 8 } }, - { name: replay_id, type: UUID, args: { schema_modifiers: [nullable] } } ] local_table_name: discover_local dist_table_name: discover_dist @@ -116,7 +115,9 @@ query_processors: column_name: tags - processor: UUIDColumnProcessor args: - columns: [event_id, trace_id, replay_id] + columns: !!set + trace_id: null + event_id: null - processor: HexIntColumnProcessor args: columns: !!set diff --git a/snuba/datasets/configuration/events/storages/errors.yaml b/snuba/datasets/configuration/events/storages/errors.yaml index 8bab9f6028..e0347a8d4a 100644 --- a/snuba/datasets/configuration/events/storages/errors.yaml +++ b/snuba/datasets/configuration/events/storages/errors.yaml @@ -279,6 +279,7 @@ allocation_policies: getsentry.tasks.backfill_grouping_records: max_threads: 2 concurrent_limit: 4 + query_processors: - processor: UniqInSelectAndHavingProcessor - processor: TupleUnaliaser @@ -302,7 +303,10 @@ query_processors: - processor: UserColumnProcessor - processor: UUIDColumnProcessor args: - columns: [event_id, primary_hash, trace_id, replay_id] + columns: !!set + event_id: null + primary_hash: null + trace_id: null - processor: HexIntColumnProcessor args: columns: !!set diff --git a/snuba/datasets/configuration/events/storages/errors_ro.yaml b/snuba/datasets/configuration/events/storages/errors_ro.yaml index 021ba96de1..68f18b7e00 100644 --- a/snuba/datasets/configuration/events/storages/errors_ro.yaml +++ b/snuba/datasets/configuration/events/storages/errors_ro.yaml @@ -277,6 +277,8 @@ allocation_policies: getsentry.tasks.backfill_grouping_records: max_threads: 2 concurrent_limit: 4 + + query_processors: - processor: UniqInSelectAndHavingProcessor - processor: TupleUnaliaser @@ -300,7 +302,10 @@ query_processors: - processor: UserColumnProcessor - processor: UUIDColumnProcessor args: - columns: [event_id, primary_hash, trace_id, replay_id] + columns: !!set + primary_hash: null + event_id: null + trace_id: null - processor: HexIntColumnProcessor args: columns: !!set diff --git a/snuba/datasets/configuration/transactions/storages/transactions.yaml b/snuba/datasets/configuration/transactions/storages/transactions.yaml index e9c1a8e59e..ba9c591be5 100644 --- a/snuba/datasets/configuration/transactions/storages/transactions.yaml +++ b/snuba/datasets/configuration/transactions/storages/transactions.yaml @@ -157,7 +157,6 @@ schema: }, { name: app_start_type, type: String }, { name: profile_id, type: UUID, args: { schema_modifiers: [nullable] } }, - { name: replay_id, type: UUID, args: { schema_modifiers: [nullable] } }, ] local_table_name: transactions_local dist_table_name: transactions_dist @@ -203,7 +202,7 @@ query_processors: trace.span_id: span_id - processor: UUIDColumnProcessor args: - columns: [event_id, trace_id, profile_id, replay_id] + columns: [event_id, trace_id, profile_id] - processor: HexIntColumnProcessor args: columns: [span_id]