Skip to content

Fix archive view column mismatch, wait_stats thread-safety, percent_complete cast, status bar color#235

Merged
erikdarlingdata merged 2 commits intodevfrom
feature/fix-archive-view-union-234
Feb 23, 2026
Merged

Fix archive view column mismatch, wait_stats thread-safety, percent_complete cast, status bar color#235
erikdarlingdata merged 2 commits intodevfrom
feature/fix-archive-view-union-234

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • UNION ALL → UNION ALL BY NAME in archive views so schema migrations don't break parquet unions (query_snapshots grid was empty)
  • Lazy<HashSet> for ignored wait types to prevent concurrent HashSet corruption when multiple servers collect in parallel
  • Convert.ToDecimal for percent_complete (dm_exec_requests returns real, not decimal)
  • ForegroundBrush for collector health status bar text (was grey, should be white like other status text)

Test plan

  • Verified UNION ALL BY NAME returns correct results with real parquet archives (152 archived rows visible)
  • Verified NULL handling for new columns from old parquet data
  • Built and launched — all collectors SUCCESS, zero errors after fix
  • Status bar text color matches other elements

Fixes #234

🤖 Generated with Claude Code

erikdarlingdata and others added 2 commits February 22, 2026 22:04
…nt_complete type cast (#234)

- UNION ALL → UNION ALL BY NAME in archive views so schema migrations
  don't break parquet unions (query_snapshots grid was empty)
- Lazy<HashSet<string>> for ignored wait types to prevent concurrent
  HashSet corruption when multiple servers collect in parallel
- Convert.ToDecimal for percent_complete (dm_exec_requests returns real,
  not decimal)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit a1a740e into dev Feb 23, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/fix-archive-view-union-234 branch February 23, 2026 21:07
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