docs(datadog): document the batches_saved queue metric - #12231
Merged
Conversation
This was referenced Jul 27, 2026
Member
Author
|
This pull request is part of a Mergify stack:
|
Contributor
Merge Protections🟢 All 7 merge protections satisfied — ready to merge. Show 7 satisfied protections🟢 ⛓️ Depends-On RequirementsRequirement based on the presence of
🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
The metrics reference was missing `mergify.queue.skip_intermediate_results.batches_saved`, a count the engine's stats metrics pusher has been submitting alongside the rest of the `mergify.queue.*` family. It is what backs the "Batches Saved" chart on the merge queue Statistics page. Add the table row and a short paragraph explaining what it counts and that it only moves when `skip_intermediate_results` is enabled. The explanation goes in prose rather than the table cell because a metric name that long plus an inline link exceeds the 120-character line limit, and table rows cannot be wrapped. MRGFY-8213 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Change-Id: I4d81d0e0cc1ddea0fde7ca8a469bb99570a7db11
jd
force-pushed
the
devs/jd/jd/mrgfy-8213-product-change-watcher-docs-follow-the-product/document-batches-saved-queue-metric--4d81d0e0
branch
from
July 27, 2026 14:30
f30ab27 to
eae3547
Compare
jd
force-pushed
the
devs/jd/jd/mrgfy-8213-product-change-watcher-docs-follow-the-product/partition-rules-removed-deprecated--5a5aeb9e
branch
from
July 27, 2026 14:30
214b594 to
6a5cbfd
Compare
Member
Author
Revision history
|
jd
marked this pull request as ready for review
July 28, 2026 08:17
Base automatically changed from
devs/jd/jd/mrgfy-8213-product-change-watcher-docs-follow-the-product/partition-rules-removed-deprecated--5a5aeb9e
to
main
July 28, 2026 08:26
mergify
Bot
force-pushed
the
devs/jd/jd/mrgfy-8213-product-change-watcher-docs-follow-the-product/document-batches-saved-queue-metric--4d81d0e0
branch
from
July 28, 2026 08:26
eae3547 to
67c125e
Compare
kozlek
approved these changes
Jul 28, 2026
remyduthu
approved these changes
Jul 28, 2026
Contributor
Merge Queue Status
This pull request spent 2 minutes 37 seconds in the queue, including 2 minutes 4 seconds running CI. Required conditions to merge
|
55 tasks
mergify
Bot
deleted the
devs/jd/jd/mrgfy-8213-product-change-watcher-docs-follow-the-product/document-batches-saved-queue-metric--4d81d0e0
branch
July 28, 2026 09:19
55 tasks
mergify Bot
pushed a commit
that referenced
this pull request
Jul 28, 2026
"Monitoring your Merge Queue" listed six freeform metrics that no longer line up with the dashboard. The Statistics page groups its widgets into Queue Health, Throughput and Performance under a row of headline figures, and gained two things this quarter the page never mentioned: the Time Breakdown widget's Avg/P95/Max selector, and the Batches Saved chart. Rewrite the section to follow the page: every widget by its real name, what it tells you, and the reference-line semantics for Max Queue Size (max batch size times max_parallel_checks) and Average Batch Size (the largest configured batch_size). The Time Breakdown entry explains the selector and why the center total only appears for the average: p95 and max of the components are not additive. Also correct the retention claim while it moves. The "Past 3 months" preset spans roughly 91 to 92 days against 90 days of queue-event retention, so its oldest day or two come back empty; the page promised that every preset returned complete data. MRGFY-8213 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Depends-On: #12231
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The metrics reference was missing
mergify.queue.skip_intermediate_results.batches_saved, a count the engine'sstats metrics pusher has been submitting alongside the rest of the
mergify.queue.*family. It is what backs the "Batches Saved" chart on themerge queue Statistics page.
Add the table row and a short paragraph explaining what it counts and that it
only moves when
skip_intermediate_resultsis enabled. The explanation goes inprose rather than the table cell because a metric name that long plus an inline
link exceeds the 120-character line limit, and table rows cannot be wrapped.
MRGFY-8213
Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com
Depends-On: #12230