Skip to content

Fix drill-down windows: inline plan XML, time range, aggregation#189

Merged
erikdarlingdata merged 1 commit intodevfrom
fix/drill-down-fixes
Feb 20, 2026
Merged

Fix drill-down windows: inline plan XML, time range, aggregation#189
erikdarlingdata merged 1 commit intodevfrom
fix/drill-down-fixes

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Remove query_plan_text/query_plan from drill-down SELECTs (was loading 122 MB inline, hanging the app)
  • Add on-demand plan fetch via Download button for all three drill-down types
  • Pass parent view's time range through to drill-down queries so they respect the selected hour/date filter
  • Aggregate query_stats drill-down by collection_time with MAX for cumulative counters (matches report.query_stats_summary)
  • Aggregate query_store drill-down by collection_time + plan_id with weighted averages
  • Remove broken 7-day DATEADD filter from PR Add 7-day time filter to drill-down queries (#165) #179 that prevented drill-downs from returning data
  • Fix Download button: RowHeight 30->35, remove stale IsEnabled binding
  • Fix DefaultTraceContent/CurrentConfigContent StaticResource -> DynamicResource
  • Add upgrade script for memory_stats columns (1.2.0 -> 1.3.0)

Test plan

  • Open Query Stats tab, double-click a row — drill-down loads, respects time range
  • Open Query Store tab, double-click a row — drill-down loads with per-plan chart lines
  • Open Procedure Stats tab, double-click a row — drill-down loads
  • Click Download button in any drill-down — fetches plan on demand, save dialog works
  • Switch time range buttons (1h, 8h, 7d) and verify drill-down row counts change accordingly
  • Verify execution counts in drill-down are consistent with main view

🤖 Generated with Claude Code

…ing, aggregate by collection_time

- Remove query_plan_text/query_plan from drill-down SELECT queries (was loading 122 MB inline, hanging the app)
- Add on-demand plan fetch methods for all three drill-down types
- Pass parent view's time range (hoursBack/fromDate/toDate) through to drill-down queries
- Aggregate query_stats drill-down by collection_time with MAX for cumulative counters (matches report.query_stats_summary pattern)
- Aggregate query_store drill-down by collection_time + plan_id with weighted averages
- Remove broken 7-day DATEADD filter from PR #179 that prevented drill-downs from returning data
- Fix Download button: increase RowHeight 30->35, remove stale IsEnabled binding, async on-demand fetch
- Fix DefaultTraceContent/CurrentConfigContent StaticResource -> DynamicResource for theme support
- Add upgrade script for memory_stats columns (1.2.0 -> 1.3.0)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit a252245 into dev Feb 20, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the fix/drill-down-fixes branch February 20, 2026 03:31
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