Skip to content

SVLS-8260 Add stats flusher to continuous mode#977

Merged
dd-mergequeue[bot] merged 1 commit intomainfrom
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 intomainfrom
dd/SVLS-8260-stats-flush-continuous

Conversation

@litianningdatadog
Copy link
Copy Markdown
Contributor

@litianningdatadog litianningdatadog commented Jan 6, 2026

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

Co-authored-by: litianningdatadog <223506508+litianningdatadog@users.noreply.github.com>
@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
Copy link
Copy Markdown
Contributor

@duncanista duncanista left a comment

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
Copy link
Copy Markdown

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

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