From 6067fc8e4cbd290861117c93a438cb957be0a3e7 Mon Sep 17 00:00:00 2001 From: Lyn Nagara Date: Thu, 13 Oct 2022 11:59:08 -0700 Subject: [PATCH] ref: Remove default mapping of "transaction" topic to "events" (#3250) Should be (roughly) synchronized with https://github.com/getsentry/sentry/pull/39988 which will send events to the new topic. This would only affect self hosted and dev users who are using the Kafka backend. All Sentry prod environments have already been migrated via other methods to ensure zero downtime during the transition. --- snuba/settings/__init__.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/snuba/settings/__init__.py b/snuba/settings/__init__.py index cab556b6cbf..ac28465c82d 100644 --- a/snuba/settings/__init__.py +++ b/snuba/settings/__init__.py @@ -185,11 +185,7 @@ class RedisClusters(TypedDict): } # Mapping of default Kafka topic name to custom names -KAFKA_TOPIC_MAP: Mapping[str, str] = { - # TODO: Remove once we are done splitting transactions from the shared events topic - "transactions": "events", - "snuba-transactions-commit-log": "snuba-commit-log", -} +KAFKA_TOPIC_MAP: Mapping[str, str] = {} # Mapping of default Kafka topic name to broker config KAFKA_BROKER_CONFIG: Mapping[str, Mapping[str, Any]] = {}