Skip to content

feat(spans): Optionally produce Span V2 Kafka messages#5151

Merged
jjbayer merged 44 commits intomasterfrom
feat/spans-produce-compat
Sep 23, 2025
Merged

feat(spans): Optionally produce Span V2 Kafka messages#5151
jjbayer merged 44 commits intomasterfrom
feat/spans-produce-compat

Conversation

@jjbayer
Copy link
Copy Markdown
Member

@jjbayer jjbayer commented Sep 18, 2025

We will migrate relay and span consumers to use a superset of the Span V2 schema as Kafka schema. This PR makes a start by using the new format for standalone spans. The new functionality is guarded by a global config option.

Next steps after this PR:

  1. Roll out standalone spans to 1%, see if consumer breaks.
  2. Roll out standalone spans to 100%.
  3. Roll out extracted spans to 1%.
  4. Roll out all spans to 100%.

ref: INGEST-531

@linear
Copy link
Copy Markdown

linear bot commented Sep 18, 2025

@jjbayer jjbayer marked this pull request as ready for review September 19, 2025 06:49
@jjbayer jjbayer requested a review from a team as a code owner September 19, 2025 06:50
@jjbayer jjbayer changed the title Add ability to produce Span V2 Kafka messages feat(spans): Optionally produce Span V2 Kafka messages Sep 22, 2025
@jjbayer jjbayer marked this pull request as draft September 22, 2025 06:11
@jjbayer jjbayer marked this pull request as ready for review September 22, 2025 07:04
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@jjbayer jjbayer enabled auto-merge September 22, 2025 07:41
@jjbayer jjbayer added this pull request to the merge queue Sep 23, 2025
Merged via the queue into master with commit bd2bd53 Sep 23, 2025
28 checks passed
@jjbayer jjbayer deleted the feat/spans-produce-compat branch September 23, 2025 06:21
github-merge-queue bot pushed a commit that referenced this pull request Sep 25, 2025
Direct follow-up to #5151: Also
produce spans that are extracted from transactions in the new format.

ref: #INGEST-531
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.

2 participants