Skip to content

Fix first-collection spike for query and procedure stats charts#491

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/fix-query-proc-spike-482
Mar 9, 2026
Merged

Fix first-collection spike for query and procedure stats charts#491
erikdarlingdata merged 1 commit intodevfrom
feature/fix-query-proc-spike-482

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Follow-up to Fix PerfMon first-value spike that skews graphs #483, reported by @dphugo in PR comment
  • Passes baselineOnly: true to CalculateDelta for query stats (8 counters) and procedure stats (6 counters)
  • First collection now stores the baseline and returns 0 instead of the raw cumulative value
  • Fixes the spike on Queries > Performance Trends charts

Test plan

  • Start Lite with a fresh DuckDB (or clear query/proc stats tables)
  • Verify Performance Trends chart starts flat, no initial spike
  • Verify subsequent collections show correct delta values

🤖 Generated with Claude Code

Pass baselineOnly: true to CalculateDelta for query_stats and proc_stats
collectors so the first cumulative value is treated as a baseline rather
than plotted as a massive spike on Performance Trends charts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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