Skip to content

Lite incremental collection: query store, CPU dedup, deadlock timing (#147)#152

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/lite-incremental-collection
Feb 19, 2026
Merged

Lite incremental collection: query store, CPU dedup, deadlock timing (#147)#152
erikdarlingdata merged 1 commit intodevfrom
feature/lite-incremental-collection

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Query store collector fetches only new runtime_stats intervals since last collection instead of scanning 60-min window with CTE/GROUP BY every cycle
  • CPU collector deduplicates ring buffer samples client-side (~1 row/cycle instead of 60)
  • Deadlock collector moves XML parsing out of DuckDB timing block for accurate metrics
  • Shared GetLastCollectedTimeAsync helper extracts watermark lookup pattern

Test plan

  • dotnet build -c Debug — 0 warnings, 0 errors
  • Launch Lite, connect to sql2022, watch collection_log timings over 2-3 cycles
  • Verify query_store SQL timing drops from ~2300ms to ~100-200ms
  • Verify cpu_utilization inserts ~1 row/cycle instead of 60
  • Verify deadlock DuckDB timing drops from ~677ms to single-digit ms

🤖 Generated with Claude Code

…fix (#147)

Query store collector now fetches only new runtime_stats intervals since
last collection instead of scanning a 60-minute window with CTE/GROUP BY
every cycle. CPU collector deduplicates ring buffer samples client-side
to insert ~1 row/cycle instead of 60. Deadlock collector moves XML
parsing out of DuckDB timing block for accurate metrics. Shared
GetLastCollectedTimeAsync helper extracts watermark lookup pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit cf56b17 into dev Feb 19, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/lite-incremental-collection branch February 19, 2026 16:28
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