Skip to content

ref(spans): Run process-segments as a task - #4488

Merged
untitaker merged 1 commit into
masterfrom
markus/process-segments-taskbroker
Aug 25, 2026
Merged

ref(spans): Run process-segments as a task#4488
untitaker merged 1 commit into
masterfrom
markus/process-segments-taskbroker

Conversation

@untitaker

Copy link
Copy Markdown
Member

Segments are already processed by the process_segment task in production.
This sets spans.buffer.process-segments-task-rollout-rate to 1.0 and drops the
dedicated process-segments Kafka consumer.

The option is set here rather than relying on the default in sentry because
self-hosted and sentry are released on different cadences, so a default change
in sentry would not take effect until the next release.

Sentry counterpart: getsentry/sentry#122555

ref STREAM-1568

Segments are already processed by the `process_segment` task in production.
Set `spans.buffer.process-segments-task-rollout-rate` to 1.0 and drop the
dedicated `process-segments` Kafka consumer.

The option is set here rather than relying on the default in sentry because
self-hosted and sentry are released on different cadences, so a default change
in sentry would not take effect until the next release.

ref STREAM-1568
@linear-code

linear-code Bot commented Aug 25, 2026

Copy link
Copy Markdown

STREAM-1568

untitaker added a commit to getsentry/sentry that referenced this pull request Aug 25, 2026
Segments are already processed by the `process_segment` task in
production.
This flips the default of
`spans.buffer.process-segments-task-rollout-rate` to
1.0 and drops the dedicated `process-segments` Kafka consumer from
devservices
and the devserver, so devenv matches production.

The flusher spawns the task directly, so no raw-mode taskbroker topic is
involved.

Self-hosted counterpart:
getsentry/self-hosted#4488

ref STREAM-1568
@untitaker
untitaker merged commit 4406a7f into master Aug 25, 2026
21 checks passed
@untitaker
untitaker deleted the markus/process-segments-taskbroker branch August 25, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants