Skip to content

ref(options): Remove spans.buffer.done-flush-conditional-zrem option#111613

Merged
untitaker merged 1 commit into
masterfrom
untitaker/remove-done-flush-conditional-zrem-option
Mar 26, 2026
Merged

ref(options): Remove spans.buffer.done-flush-conditional-zrem option#111613
untitaker merged 1 commit into
masterfrom
untitaker/remove-done-flush-conditional-zrem-option

Conversation

@untitaker
Copy link
Copy Markdown
Member

Summary

  • Make the conditional cleanup (race condition fix) in span buffer's done_flush_segments always-on
  • Remove the spans.buffer.done-flush-conditional-zrem option and old unconditional code path
  • The two-phase conditional cleanup (ZREM + data deletion) is now the only behavior

Refs STREAM-771

@untitaker untitaker requested review from a team as code owners March 26, 2026 13:29
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Mar 26, 2026

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 26, 2026
Make the conditional cleanup (race condition fix) in span buffer's
done_flush_segments always-on and remove the old unconditional code path.

Refs STREAM-771
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@untitaker untitaker force-pushed the untitaker/remove-done-flush-conditional-zrem-option branch from c3acd6a to 8eb6afe Compare March 26, 2026 13:31
@untitaker untitaker merged commit a42668e into master Mar 26, 2026
72 checks passed
@untitaker untitaker deleted the untitaker/remove-done-flush-conditional-zrem-option branch March 26, 2026 16:15
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 11, 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