Skip to content

ref(spans): Add additional tags to usage/count per root - #5511

Merged
Dav1dde merged 8 commits into
masterfrom
dav1d/span-metrics-more-tags
Jan 8, 2026
Merged

ref(spans): Add additional tags to usage/count per root#5511
Dav1dde merged 8 commits into
masterfrom
dav1d/span-metrics-more-tags

Conversation

@Dav1dde

@Dav1dde Dav1dde commented Jan 5, 2026

Copy link
Copy Markdown
Member

Depends on: getsentry/sentry#105630
Refs: INGEST-689

Adds has_transaction to all spans which still have a transaction which is ingested and therefor also have transaction metrics. This will be later the indicator used in consumers to not double account for transactions.

Adds is_segment to usage and count per root metrics used for dynamic sampling calculations.

@Dav1dde
Dav1dde force-pushed the dav1d/span-metrics-more-tags branch from 8293246 to 95f0330 Compare January 5, 2026 16:36
@Dav1dde Dav1dde self-assigned this Jan 5, 2026
@Dav1dde
Dav1dde marked this pull request as ready for review January 7, 2026 14:16
@Dav1dde
Dav1dde requested a review from a team as a code owner January 7, 2026 14:16
@Dav1dde
Dav1dde force-pushed the dav1d/span-metrics-more-tags branch from e83993a to c47df01 Compare January 7, 2026 14:20
Comment thread relay-server/src/processing/spans/dynamic_sampling.rs
Comment thread relay-dynamic-config/src/defaults.rs
Comment thread relay-server/src/metrics_extraction/event.rs
Comment thread relay-server/src/processing/spans/dynamic_sampling.rs Outdated
@linear

linear Bot commented Jan 8, 2026

Copy link
Copy Markdown

Comment thread relay-server/src/metrics_extraction/event.rs
@Dav1dde
Dav1dde added this pull request to the merge queue Jan 8, 2026
Merged via the queue into master with commit f4d6c62 Jan 8, 2026
29 checks passed
@Dav1dde
Dav1dde deleted the dav1d/span-metrics-more-tags branch January 8, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants