Skip to content

Document batching, PFD, and parallel-queue interactions#56

Merged
samgutentag merged 1 commit into
mainfrom
sam-gutentag/batching-pfd
Jun 1, 2026
Merged

Document batching, PFD, and parallel-queue interactions#56
samgutentag merged 1 commit into
mainfrom
sam-gutentag/batching-pfd

Conversation

@samgutentag
Copy link
Copy Markdown
Member

Summary

  • Add a new "How batching interacts with parallel queues and PFD" section to merge-queue/optimizations/batching.mdx covering: batching as a lane, PR batch eligibility, ALL-keyword non-serialization, transitive dependents, bisection splits in half, PFD's downstream-PR delay, optimistic-off batch-removal restarts, and MQ-only failures not feeding flakiness signals today.
  • Cross-link from pending-failure-depth.mdx (Next Steps) and parallel-queues/index.mdx (Related) into the new section.
  • Link to the webhooks reference for the event-side view of pending-failure batch events.

Why

Sourced from customer feedback mining (cluster batching-and-pending-failure-depth, verdict partial, 7 pairs across 4 customers). Existing pages explain batching, PFD, and bisection individually; customers hit confusion when these interact (e.g., the PFD-delay-when-downstream-still-testing case, ALL-keyword not serializing parallel PRs, optimistic-off causing batch-removal restarts). This PR addresses the cross-interaction framing gap the verifier called out, without re-documenting the individual features.

Items flagged for review

  • PFD downstream-PR delay example — verify wording matches eng intent. The example states that a newly-arrived PR-C with overlapping targets extends the wait before PR-A is kicked. Confirm "shares impacted targets" is the right gate (vs. all successors regardless of target overlap).
  • ALL keyword serialization claim — confirm the wording on concurrency interaction. The section says downstream PRs without shared targets can test in parallel behind an ALL-impacting PR "up to your testing concurrency." Verify this matches the parallel-mode behavior exactly.
  • "Open to adding number of conclusions alongside depth" — the source thread mentions Phil being open to this. The doc reframes it as "active area of design — share your use case with support" to avoid promising a roadmap item. Confirm the soft framing is acceptable.
  • Bisection concurrency framing — claim is that "setting it equal to your batch size is more than you need" because bisection is logarithmic. Worth a sanity check against current bisection implementation.
  • Optimistic-off restart wording — the doc attributes batch-removal restarts to optimistic merging being disabled. Source confirms this for one case; verify it's the general rule.
  • MQ-only failures and flakiness — claim is that MQ-only failures and bisection runs aren't fed into Flaky Tests "today." Verify this is still accurate and that the soft "yet"-style framing is right.
  • Cross-link to webhooks anchor — currently links to ../webhooks without anchor since PR Rewrite webhooks reference with event lifecycle and v1/v2 schema notes #52 (webhook lifecycle reference) is not yet merged. Once Rewrite webhooks reference with event lifecycle and v1/v2 schema notes #52 lands, update to ../webhooks#pending-failure-batch (or whatever the final anchor is).

Customer signal

@samgutentag samgutentag added the needs review PR sourced from customer-feedback-mining; needs human scrutiny for accuracy before merge label May 20, 2026
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 20, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
trunk 🟢 Ready View Preview May 20, 2026, 11:52 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@samgutentag
Copy link
Copy Markdown
Member Author

samgutentag commented May 26, 2026

Verification status (2026-05-29): live

Verified: customers can use this. Ready to publish.

  • Flag state: no feature flag involved; LaunchDarkly not consulted (nothing to read)
  • Eng PR: none (customer feedback-sourced batching/PFD interaction docs)
  • Flag: none
  • Signals: No engineering dependency and no feature flag. This PR documents existing interactions between batching, pending failure depth, and parallel queues, sourced from 7 customer pairs across 4 channels. All described mechanics (batch-as-lane, ALL-keyword serialization, bisection splits, PFD downstream-PR delay, optimistic-off restarts) are currently-shipped behaviors.

Ready to publish after editorial and eng review of flagged items.


Generated by Claude Code

@samgutentag samgutentag added ready to merge Verify docs PR: customers can use this. Ready to publish. and removed needs review PR sourced from customer-feedback-mining; needs human scrutiny for accuracy before merge labels May 26, 2026
Copy link
Copy Markdown
Member Author

Verification status (May 31, 2026): live

Batching / PFD docs cover existing, shipped mechanics — no flag or eng dependency.

  • Flag state: not consulted
  • Eng PR: none
  • Flag: none
  • Signals: documents existing batching and parallel-flaky-detection behavior already available to all customers

Next action: merge this PR.


Generated by Claude Code

Copy link
Copy Markdown
Member Author

Docs Verify Sweep · June 1, 2026 · ✅ live

Feature is live in production. Ready to merge.

Unchanged from previous sweep (May 31, 2026).


Generated by Claude Code

@samgutentag samgutentag merged commit a2fc6be into main Jun 1, 2026
3 checks passed
@samgutentag samgutentag deleted the sam-gutentag/batching-pfd branch June 1, 2026 21:04
@samgutentag samgutentag restored the sam-gutentag/batching-pfd branch June 1, 2026 21:16
samgutentag added a commit that referenced this pull request Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Verify docs PR: customers can use this. Ready to publish.

Development

Successfully merging this pull request may close these issues.

1 participant