New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplification and cleanup of decompression planning #6731
Conversation
Add ANALYZE after compression of `metrics_compressed` in shared test setup.
We don't account for them at all currently, which leads to weird planning behavior, like the filters not influencing the position of relations in join.
…ially_compressed-*
…ially_compressed-*
…ially_compressed-*
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6731 +/- ##
==========================================
+ Coverage 80.06% 80.99% +0.92%
==========================================
Files 190 191 +1
Lines 37181 36337 -844
Branches 9450 9432 -18
==========================================
- Hits 29770 29430 -340
- Misses 2997 3159 +162
+ Partials 4414 3748 -666 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. However, I recommend to change the commit message, since the plans are changed, it is not only a cosmetic change.
This PR renames some things in preparation to cost changes from #6550
Also simplifies path creation for partial chunks, improving some plans (now index scans are used instead of sort).
Disable-check: force-changelog-file