Skip to content

Reduce first-run lookback to 1 hour for all collectors#340

Merged
erikdarlingdata merged 1 commit intodevfrom
fix/first-run-lookback
Feb 27, 2026
Merged

Reduce first-run lookback to 1 hour for all collectors#340
erikdarlingdata merged 1 commit intodevfrom
fix/first-run-lookback

Conversation

@erikdarlingdata
Copy link
Owner

Summary

Collectors changed

Collector Before After
query_stats All DMV data 1 hour
query_store 3 days 1 hour
procedure_stats All DMV data 1 hour
cpu_utilization_stats 7 days 1 hour
blocked_processes 3 days 1 hour
deadlock_xml 3 days 1 hour
system_health_wrapper 3 days 1 hour
default_trace All available 1 hour

Test plan

  • Clean install on sql2016 — all 51 scripts pass, zero errors
  • All 48 collectors run successfully after install

🤖 Generated with Claude Code

…335)

All 8 collectors with first-run backfill logic now look back 1 hour
instead of 3-7 days (or unlimited). The aggressive lookback was a
development convenience that becomes catastrophic on large environments
— particularly query_store_collector with XML plan decompression on
multi-GB Query Store databases, causing timeout→rollback→retry loops.

Changed collectors:
- 08_collect_query_stats: all DMV data → 1 hour
- 09_collect_query_store: 3 days → 1 hour
- 10_collect_procedure_stats: all DMV data → 1 hour
- 18_collect_cpu_utilization_stats: 7 days → 1 hour
- 22_collect_blocked_processes: 3 days → 1 hour
- 24_collect_deadlock_xml: 3 days → 1 hour
- 28_collect_system_health_wrapper: 3 days → 1 hour
- 29_collect_default_trace: all available → 1 hour

Clean install verified on sql2016 with zero errors.

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