Skip to content

Conversation

@AhmedSoliman
Copy link
Contributor

@AhmedSoliman AhmedSoliman commented Nov 25, 2025

Introduces a small set of cumulative metrics from the vqueue scheduler to help visualize:

  • The rate of items in scheduler decisions broken down by action
  • The enqueue rate (in waiting inbox)
  • Counters for confirmed/rejected items (confirmed items is low signal and we can remove it)
  • The cumulative wait time on invoker capacity. The absolute value itself is not interesting, but the growth in the value indicates active throttling based on invoker concurrency.

Stack created with Sapling. Best reviewed with ReviewStack.

…nized with eligibility_at

This changeset makes sure that we have a reliable datum point that's shared between the delayed queue internal timer wheel and the scheduler.
Additionally, it ensures that the scheduler can create `Instant` values that are synchronized with the datum point acquired from UniqueTimestamp.

This fixes the issue introduced in changeset 6e65909 where we would become eligible slightly before the actual eligibility time, causing supurious rescheduling.
Introduces a small set of cumulative metrics from the vqueue scheduler to help visualize:
- The rate of items in scheduler decisions broken down by action
- The enqueue rate (in waiting inbox)
- Counters for confirmed/rejected items (confirmed items is low signal and we can remove it)
- The cumulative wait time on invoker capacity. The absolute value itself is not interesting, but the growth in the value indicates active throttling based on invoker concurrency.
@github-actions
Copy link

github-actions bot commented Nov 25, 2025

Test Results

  7 files  ±0    7 suites  ±0   3m 12s ⏱️ +2s
 47 tests ±0   47 ✅ ±0  0 💤 ±0  0 ❌ ±0 
200 runs  ±0  200 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 0d5c508. ± Comparison against base commit c05c1de.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants