Skip to content

Commit

Permalink
mypy
Browse files Browse the repository at this point in the history
  • Loading branch information
volokluev committed Sep 12, 2024
1 parent f2ad39f commit 21592e7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions snuba/web/rpc/common.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from typing import Final, Mapping, Set
from typing import Final, Mapping, Sequence, Set

from sentry_protos.snuba.v1alpha.request_common_pb2 import RequestMeta
from sentry_protos.snuba.v1alpha.trace_item_attribute_pb2 import (
Expand Down Expand Up @@ -134,7 +134,7 @@ def attribute_key_to_expression(attr_key: AttributeKey) -> Expression:


def apply_virtual_columns(
query: Query, virtual_column_contexts: list[VirtualColumnContext]
query: Query, virtual_column_contexts: Sequence[VirtualColumnContext]
) -> None:
"""Injects virtual column mappings into the clickhouse query. Works with NORMALIZED_COLUMNS on the table or
dynamic columns in attr_str
Expand Down Expand Up @@ -175,7 +175,7 @@ def transform_expressions(expression: Expression) -> Expression:
if not isinstance(expression, SubscriptableReference):
return expression

if not expression.column != "attr_str":
if not expression.column.column_name != "attr_str":
return expression
context = mapped_column_to_context.get(str(expression.key.value))
if context:
Expand Down

0 comments on commit 21592e7

Please sign in to comment.