docs: combined concurrency limits and queue gates - #4853
Conversation
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
WalkthroughThe OpenAPI specification adds combined concurrency override and reset endpoints. It documents combined concurrency state on Merge Risk: ⚪ Minimal · up to This change documents combined queue concurrency APIs, configuration, and usage patterns. No current merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description accurately summarizes the documentation changes, but it does not follow the repository template. It omits the required checklist, testing details, changelog, screenshots section, and issue-closing line. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
05bef1a to
7998d68
Compare
af1ed4f to
ab4c607
Compare
e73840d to
34b2cfb
Compare
@trigger.dev/build
trigger.dev
@trigger.dev/core
@trigger.dev/python
@trigger.dev/react-hooks
@trigger.dev/redis-worker
@trigger.dev/rsc
@trigger.dev/schema-to-json
@trigger.dev/sdk
commit: |
34b2cfb to
947879a
Compare
947879a to
2e0ae55
Compare
2e0ae55 to
1772166
Compare
99fc054 to
7cd93f1
Compare
7cd93f1 to
2250f8e
Compare
2250f8e to
b70f42a
Compare
b70f42a to
b4d89eb
Compare
7038e96 to
1d2ac16
Compare
1d2ac16 to
f9f6ea8
Compare
f9f6ea8 to
7d3d148
Compare
7d3d148 to
d8b396d
Compare
d8b396d to
7d38a93
Compare
7d38a93 to
98b74d9
Compare
98b74d9 to
a615cbb
Compare
a615cbb to
40f9d4f
Compare
40f9d4f to
b5f2d4f
Compare
b5f2d4f to
b59bdb0
Compare
…body required The gates intro promised a task-wide cap the keyed example does not deliver (a concurrencyKey splits the home queue per key); the combined object is emitted on every queue with null fields rather than omitted; and both reset endpoints reject a zero-length body, so the body is required. Also restores the example that drifted off overriddenAt.
A Use cases index links each goal to its section, the multi-queue section names the home queue and gate concepts once and gives each pattern its own worked example (per-tenant cap across tasks, global cap for a shared resource via a combined-only queue, pinned-key shared pool), and the per-key-except-combined rule gets a warning callout. Folds in the simplified wording and removes self-hosting notes.
The combined limit only counts keyed runs, so the shared-resource example now shows the keyed trigger and warns that keyless runs bypass the cap, pointing those cases at the pinned-key pool.
b59bdb0 to
bf2e961
Compare
Summary
Documents the queue concurrency features shipping in this stack: the
combinedConcurrencyLimitqueue option that caps a keyed queue across all of itsconcurrencyKeyvalues, queue gates (arrayqueuesyntax for holding a slot in more than one queue), and the combined override/reset SDK methods and endpoints.The concurrency guide gains sections on combined limits and gates with self-hosting notes for the server flags, the OpenAPI spec gains the two combined endpoints and the
concurrency.combinedresponse field, and the management reference gains pages for both new endpoints.Stacked on #4830.