Skip to content

Add avg ms per wait chart toggle (#22)#54

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/22-avg-ms-per-wait
Feb 16, 2026
Merged

Add avg ms per wait chart toggle (#22)#54
erikdarlingdata merged 1 commit intodevfrom
feature/22-avg-ms-per-wait

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Added Metric dropdown to wait stats chart in both Dashboard and Lite
  • Two modes: "Wait Time (ms/sec)" (default) and "Avg Wait Time (ms/wait)"
  • Avg mode computes delta_wait_time_ms / delta_waiting_tasks per collection interval
  • Hover tooltips and Y-axis labels update to match the selected metric
  • Lite WaitStatsRow gains AvgWaitMsPerTask computed property for future grid use
  • No schema changes — waiting_tasks_count was already collected in both apps

Why

Total wait time (ms/sec) answers "where is time going?" but avg ms/wait answers "which waits are individually the slowest?" — e.g., CXPACKET dominates total time but each wait is trivial, while RESOURCE_SEMAPHORE may be low total but each wait is brutal.

Test plan

  • Lite: toggle between ms/sec and ms/wait — chart redraws, Y-axis and tooltip units update
  • Dashboard: same toggle behavior on Wait Stats Detail chart
  • Both build with 0 errors
  • Values look reasonable (high-volume waits show low avg, low-volume waits show high avg)

Closes #22

🤖 Generated with Claude Code

Wait stats chart now has a Metric dropdown to switch between:
- Wait Time (ms/sec): existing per-second rate (default)
- Avg Wait Time (ms/wait): delta_wait_time_ms / delta_waiting_tasks

Shows which waits are individually painful vs just high volume.
Both Dashboard and Lite get the toggle, hover tooltip unit updates
to match, and Lite summary grid gains AvgWaitMsPerTask property.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit ee3fcb2 into dev Feb 16, 2026
1 check passed
@erikdarlingdata erikdarlingdata deleted the feature/22-avg-ms-per-wait branch February 20, 2026 13:20
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.

1 participant