Skip to content

Commit

Permalink
fix(span): ADd replacement column for transaction.id (#78862)
Browse files Browse the repository at this point in the history
- Replacing `transaction.id` and `span.id` for `transaction.span_id`.
Leaving them in cause the frontend might still be using them
  • Loading branch information
wmak authored Oct 9, 2024
1 parent 3c7fbcc commit 60ef0f2
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/sentry/utils/snuba.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ def log_snuba_info(content):
"transaction": "segment_name",
"transaction.id": "transaction_id",
"segment.id": "segment_id",
"transaction.span_id": "segment_id",
"transaction.op": "transaction_op",
"user": "user",
"user.id": "sentry_tags[user.id]",
Expand Down Expand Up @@ -198,7 +199,11 @@ def log_snuba_info(content):
"timestamp": "timestamp",
"trace": "trace_id",
"transaction": "segment_name",
# `transaction.id` and `segment.id` is going to be replaced by `transaction.span_id` please do not use
# transaction.id is "wrong", its pointing to segment_id to return something for the transistion, but represents the
# txn event id(32 char uuid). EAP will no longer be storing this.
"transaction.id": "segment_id",
"transaction.span_id": "segment_id",
"transaction.method": "attr_str[transaction.method]",
"is_transaction": "is_segment",
"segment.id": "segment_id",
Expand Down

0 comments on commit 60ef0f2

Please sign in to comment.