From cf5aa5b2d894bd84e454ff889b3dd4cf7f81163e Mon Sep 17 00:00:00 2001 From: Lyn Nagara Date: Fri, 13 Nov 2020 14:54:26 -0800 Subject: [PATCH] feat(errors): Add event ID column processing (#1488) Like Transactions and Discover, we also need to perform event_id column processing for errors since these are also stored as UUIDs. --- snuba/datasets/storages/errors_common.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/snuba/datasets/storages/errors_common.py b/snuba/datasets/storages/errors_common.py index d1f851bdba..90ca42a732 100644 --- a/snuba/datasets/storages/errors_common.py +++ b/snuba/datasets/storages/errors_common.py @@ -11,10 +11,10 @@ ) from snuba.clickhouse.columns import SchemaModifiers as Modifiers from snuba.datasets.errors_replacer import ReplacerState +from snuba.datasets.storages.event_id_column_processor import EventIdColumnProcessor from snuba.datasets.storages.processors.replaced_groups import ( PostReplacementConsistencyEnforcer, ) - from snuba.query.conditions import ConditionFunctions, binary_condition from snuba.query.expressions import Column, Literal from snuba.query.processors.arrayjoin_keyvalue_optimizer import ( @@ -136,6 +136,7 @@ project_column="project_id", replacer_state_name=ReplacerState.ERRORS, ), MappingColumnPromoter(mapping_specs={"tags": promoted_tag_columns}), + EventIdColumnProcessor(), ArrayJoinKeyValueOptimizer("tags"), PrewhereProcessor(), ]