Skip to content

SVLS-8260 Add stats flusher to continuous mode#977

Merged
dd-mergequeue[bot] merged 1 commit into
mainfrom
dd/SVLS-8260-stats-flush-continuous
Jan 9, 2026
Merged

SVLS-8260 Add stats flusher to continuous mode#977
dd-mergequeue[bot] merged 1 commit into
mainfrom
dd/SVLS-8260-stats-flush-continuous

Conversation

@litianningdatadog

@litianningdatadog litianningdatadog commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

PR by Bits
View Dev Agent Session

You can ask for changes by mentioning @DataDog in a comment.

Feedback (especially what can be better) welcome in #code-gen-feedback!


Overview

  • align continuous flush path to include stats
  • track stats flush handles to prevent overlaps and redrive cleanly
  • add unit test for stats handle tracking

@litianningdatadog
litianningdatadog force-pushed the dd/SVLS-8260-stats-flush-continuous branch from 908db95 to 319d357 Compare January 7, 2026 17:22
@litianningdatadog
litianningdatadog marked this pull request as ready for review January 8, 2026 19:09
@litianningdatadog
litianningdatadog requested a review from a team as a code owner January 8, 2026 19:09
Comment thread bottlecap/src/bin/bottlecap/main.rs

@duncanista duncanista left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM – let's test in self monitoring in order to ensure nothing is broken

@litianningdatadog

Copy link
Copy Markdown
Contributor Author

Tested using self-monitoring stack and no issue was observed.

@litianningdatadog

Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jan 9, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-01-09 22:11:40 UTC ℹ️ Start processing command /merge


2026-01-09 22:11:45 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 1m (p90).


2026-01-09 22:36:18 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue
dd-mergequeue Bot merged commit 9abd5a1 into main Jan 9, 2026
51 checks passed
@dd-mergequeue
dd-mergequeue Bot deleted the dd/SVLS-8260-stats-flush-continuous branch January 9, 2026 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants