Fix view documentation validation - achieve 100% coverage (107 views)#8362
Merged
Fix view documentation validation - achieve 100% coverage (107 views)#8362
Conversation
…ization Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
[WIP] Fix issues in sample data extraction process
Fix election year view temp file overflow via materialized views
Feb 8, 2026
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Fix election year view temp file overflow via materialized views
Optimize database views: eliminate 50GB temp file errors and timeout via materialized views
Feb 8, 2026
…pe < characters) Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Optimize database views: eliminate 50GB temp file errors and timeout via materialized views
Fix view performance: eliminate cartesian products causing 50GB temp file errors and timeouts
Feb 8, 2026
Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Fix view performance: eliminate cartesian products causing 50GB temp file errors and timeouts
Fix 108 database views: eliminate cartesian products, add materialized views, insert test data
Feb 8, 2026
…, add deterministic ordering Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Fix 108 database views: eliminate cartesian products, add materialized views, insert test data
Fix query logic bugs in 5 database views returning empty results
Feb 8, 2026
…tgreSQL 16 database Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Fix query logic bugs in 5 database views returning empty results
Fix database view performance and query logic bugs
Feb 9, 2026
pethers
approved these changes
Feb 9, 2026
View Documentation Validation FailedThis PR modifies view-related files, and the validation has detected incomplete documentation coverage. The following 2 views are in the schema but missing from documentation: Other Views
Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging. See the workflow artifacts for the complete validation report. |
… views to documentation Co-authored-by: pethers <[email protected]>
Co-authored-by: pethers <[email protected]>
Copilot
AI
changed the title
Fix database view performance and query logic bugs
Fix view documentation validation - achieve 100% coverage (107 views)
Feb 9, 2026
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description
View documentation validation failed with 101.87% coverage (109 documented vs 107 in schema). Root cause: 4 obsolete views documented, 2 new materialized views undocumented, validation script only matched
view_prefix.Changes:
mv_annual_document_metricsandmv_annual_voting_metricsmaterialized views^### view_to^### (view_|mv_)to recognize both naming conventionsType of Change
Primary Changes
Technical Changes
Impact Analysis
Political Analysis Impact
Technical Impact
Testing
Validation Results:
Total views in schema: 107 Documented views: 107 Coverage: 100.00% Status: ✅ PASSEDDocumentation
Documentation Changes:
view_andmv_prefixesScreenshots
N/A - documentation and validation script changes only
Checklist
Additional Notes
Materialized Views Added to Documentation:
mv_annual_document_metrics: Pre-aggregates 109K document rows → 24 annual rows, eliminates >50GB temp filesmv_annual_voting_metrics: Pre-aggregates 3.5M vote rows → 24 annual rows, eliminates cartesian productsObsolete Views Removed:
view_riksdagen_ministry: Replaced in schema refactoringview_riksdagen_intelligence_dashboard: Dropped by CASCADE operationsview_riksdagen_election_year_vs_midterm: Refactored in db-changelog-1.70.xmlview_riksdagen_election_year_anomalies: Refactored in db-changelog-1.70.xmlValidation Script Enhancement:
Database uses two view naming conventions:
view_riksdagen_*(105 views)mv_*(2 views)Script now recognizes both patterns via regex
^### (view_|mv_).Security Considerations
Release Notes
Documentation:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.