Skip to content

ref(spanv2): Drop span kind as a top level field#5368

Merged
Dav1dde merged 5 commits intomasterfrom
dav1d/rm-spanv2-kind
Nov 12, 2025
Merged

ref(spanv2): Drop span kind as a top level field#5368
Dav1dde merged 5 commits intomasterfrom
dav1d/rm-spanv2-kind

Conversation

@Dav1dde
Copy link
Copy Markdown
Member

@Dav1dde Dav1dde commented Nov 11, 2025

Removes the kind as a top level field from a SpanV2 span. Instead merges it into the attribute sentry.kind where the buffer previously would extract it to.

The schema already lists the attribute as optional.

In a follow-up:

  • Remove the mapping from Sentry
  • Remove the field from the schema
  • Add sentry.kind to conventions

Refs: INGEST-631

@Dav1dde Dav1dde requested a review from a team as a code owner November 11, 2025 15:23
@Dav1dde Dav1dde force-pushed the dav1d/rm-spanv2-kind branch from ab2808e to 143d52b Compare November 11, 2025 15:24
@Dav1dde Dav1dde requested review from Lms24 and cleptric November 11, 2025 15:24
@Dav1dde Dav1dde force-pushed the dav1d/rm-spanv2-kind branch 2 times, most recently from 6f0f3de to 31c8c02 Compare November 11, 2025 15:31
@Dav1dde Dav1dde requested a review from untitaker November 11, 2025 15:31
@Dav1dde
Copy link
Copy Markdown
Member Author

Dav1dde commented Nov 11, 2025

@untitaker jfyi, removal of the top level kind field, from my scan of the code in Sentry, this is unused and only extracted into a sentry.kind attribute.

@Dav1dde Dav1dde force-pushed the dav1d/rm-spanv2-kind branch from 31c8c02 to 79946bd Compare November 11, 2025 15:36
@Dav1dde Dav1dde force-pushed the dav1d/rm-spanv2-kind branch from 79946bd to aa43157 Compare November 11, 2025 15:38
@Dav1dde Dav1dde disabled auto-merge November 12, 2025 08:16
@Dav1dde Dav1dde enabled auto-merge November 12, 2025 08:17
@linear
Copy link
Copy Markdown

linear bot commented Nov 12, 2025

@Dav1dde Dav1dde self-assigned this Nov 12, 2025
@Dav1dde Dav1dde added this pull request to the merge queue Nov 12, 2025
Merged via the queue into master with commit ddde129 Nov 12, 2025
80 of 83 checks passed
@Dav1dde Dav1dde deleted the dav1d/rm-spanv2-kind branch November 12, 2025 10:15
Dav1dde added a commit to getsentry/sentry that referenced this pull request Nov 13, 2025
The field is no longer part of the Schema, Relay extracts it to an
attribute already for Transaction and OTLP Spans.

Relay: getsentry/relay#5368
Schema: getsentry/sentry-kafka-schemas#457

No more usages of Span Kind:
```
┌─── [🐍.venv] src/sentry/spans <dav1d/rm-span-kind> 
└─ % rg kind
buffer.py
58:    * span-buf:z:* -- the actual set keys, containing span payloads. Each key contains all data for a segment. The most memory-intensive kind of key.

consumers/process_segments/enrichment.py
216:    Check the measurements on the span to determine what kind of start type the
```

Closes: INGEST-631
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants