Update src/sentry/ingest/transaction_clusterer/datasource/redis.py

jjbayer · olksdr · web-flow · commit eac9c9b6abf5 · 2023-06-22T16:05:10.000+02:00
Co-authored-by: Oleksandr &lt;1931331+olksdr@users.noreply.github.com&gt;
diff --git a/src/sentry/ingest/transaction_clusterer/datasource/redis.py b/src/sentry/ingest/transaction_clusterer/datasource/redis.py
@@ -137,7 +137,7 @@ def _should_store_transaction_name(event_data: Mapping[str, Any]) -> Optional[st
         # Relay leaves source None if it expects it to be high cardinality, (otherwise it sets it to "unknown")
         # (see https://github.com/getsentry/relay/blob/2d07bef86415cc0ae8af01d16baecde10cdb23a6/relay-general/src/store/transactions/processor.rs#L369-L373).
         #
-        # Our data show that a majority of these `None` source transactions contain slashes, so treat them as URL transactions:
+        # Our data shows that a majority of these `None` source transactions contain slashes, so treat them as URL transactions:
         source is None
         and "/" in transaction_name
     )

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ def _should_store_transaction_name(event_data: Mapping[str, Any]) -> Optional[st`
`137`	`137`	`# Relay leaves source None if it expects it to be high cardinality, (otherwise it sets it to "unknown")`
`138`	`138`	`# (see https://github.com/getsentry/relay/blob/2d07bef86415cc0ae8af01d16baecde10cdb23a6/relay-general/src/store/transactions/processor.rs#L369-L373).`
`139`	`139`	`#`
`140`		- # Our data show that a majority of these `None` source transactions contain slashes, so treat them as URL transactions:
	`140`	+ # Our data shows that a majority of these `None` source transactions contain slashes, so treat them as URL transactions:
`141`	`141`	`source is None`
`142`	`142`	`and "/" in transaction_name`
`143`	`143`	`)`