Skip to content

Commit 760133a

Browse files
committed
ref(crons): Remove decode compat code
This issue is no longer happening [SENTRY-38FS](https://sentry.sentry.io/issues/5256667522/)
1 parent 2c12dad commit 760133a

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

src/sentry/monitors/consumers/monitor_consumer.py

+2-12
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from functools import partial
1010
from typing import Literal
1111

12-
import msgpack
1312
import sentry_sdk
1413
from arroyo.backends.kafka.consumer import KafkaPayload
1514
from arroyo.processing.strategies.abstract import ProcessingStrategy, ProcessingStrategyFactory
@@ -864,11 +863,7 @@ def process_batch(executor: ThreadPoolExecutor, message: Message[ValuesBatch[Kaf
864863
assert isinstance(item, BrokerValue)
865864

866865
try:
867-
try:
868-
wrapper: IngestMonitorMessage = MONITOR_CODEC.decode(item.payload.value)
869-
except ValidationError:
870-
wrapper = msgpack.unpackb(item.payload.value)
871-
logger.exception("Failed to unpack message payload via sentry_kafka_schemas")
866+
wrapper: IngestMonitorMessage = MONITOR_CODEC.decode(item.payload.value)
872867
except Exception:
873868
logger.exception("Failed to unpack message payload")
874869
continue
@@ -913,12 +908,7 @@ def process_batch(executor: ThreadPoolExecutor, message: Message[ValuesBatch[Kaf
913908
def process_single(message: Message[KafkaPayload]):
914909
assert isinstance(message.value, BrokerValue)
915910
try:
916-
try:
917-
wrapper: IngestMonitorMessage = MONITOR_CODEC.decode(message.payload.value)
918-
except ValidationError:
919-
logger.exception("Failed to unpack message payload via sentry_kafka_schemas")
920-
wrapper = msgpack.unpackb(message.payload.value)
921-
911+
wrapper: IngestMonitorMessage = MONITOR_CODEC.decode(message.payload.value)
922912
ts = message.value.timestamp
923913
partition = message.value.partition.index
924914

0 commit comments

Comments
 (0)