Skip to content

feat(spans-buffer): Add dual write for ZSET to SET migration#106794

Merged
lvthanh03 merged 7 commits intomasterfrom
tony/dual-write-zset-set
Jan 23, 2026
Merged

feat(spans-buffer): Add dual write for ZSET to SET migration#106794
lvthanh03 merged 7 commits intomasterfrom
tony/dual-write-zset-set

Conversation

@lvthanh03
Copy link
Copy Markdown
Member

@lvthanh03 lvthanh03 commented Jan 22, 2026

Refs STREAM-712

Implements span buffer storage in regular SET and reading from SET keys. We will test the dual write mode to validate the new storage format before switching the flusher reader.

@linear
Copy link
Copy Markdown

linear Bot commented Jan 22, 2026

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 22, 2026
@lvthanh03 lvthanh03 marked this pull request as ready for review January 22, 2026 19:54
@lvthanh03 lvthanh03 requested review from a team as code owners January 22, 2026 19:54
cursor[bot]

This comment was marked as outdated.

Copy link
Copy Markdown
Member

@evanh evanh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the emit_observability_metrics function needs to be called on the set_results so we emit all the correct metrics.

Comment thread src/sentry/spans/buffer.py Outdated
Comment thread src/sentry/spans/buffer.py Outdated
Comment thread src/sentry/spans/buffer.py Outdated
Comment thread src/sentry/spans/buffer.py Outdated
Comment thread src/sentry/spans/buffer.py
Comment thread src/sentry/spans/buffer.py Outdated
Comment thread src/sentry/spans/buffer.py
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread src/sentry/spans/buffer.py
Copy link
Copy Markdown
Member

@evanh evanh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, minus the one nit.

Comment thread src/sentry/spans/buffer.py Outdated
@lvthanh03 lvthanh03 merged commit dd73f2d into master Jan 23, 2026
66 checks passed
@lvthanh03 lvthanh03 deleted the tony/dual-write-zset-set branch January 23, 2026 20:50
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants