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
Run regresscheck on PG13 #2836
Run regresscheck on PG13 #2836
Conversation
aac875d
to
6762faf
Compare
Codecov Report
@@ Coverage Diff @@
## master #2836 +/- ##
==========================================
+ Coverage 90.08% 90.24% +0.15%
==========================================
Files 212 212
Lines 34775 34725 -50
==========================================
+ Hits 31326 31336 +10
+ Misses 3449 3389 -60
Continue to review full report at Codecov.
|
55f8c10
to
2e494f8
Compare
fbd9e31
to
8b0c8f2
Compare
8b0c8f2
to
c7d0f35
Compare
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.
I compared the output files for PG12 and PG13 to see that there were only benign changes. Here's a summary:
- These tests are identical in PG12 and PG13:
custom_type
,multi_transaction_index
,parallel
,partition
,chunk_adaptive
,plan_hashagg
,query
,sql_query
- These tests have new names in the plans, but otherwise match:
delete
,plan_expand_hypertable
,plan_hypertable_inline
,update
- Test
partitionwise
have some changes in plans, please check below and verify that they are OK. - These tests have changes in error messages:
constraint
,ddl
,insert
c7d0f35
to
499e9f7
Compare
PG13 adds the relation name to constraint errors making the output differ from previous PG versions.
PG13 changes relation alias naming in EXPLAIN output making the output different from previous versions.
PG13 changes relation alias naming in EXPLAIN output making the output different from previous versions.
PG13 changes relation alias naming in EXPLAIN output making the output different from previous versions.
PG13 changes the error message on constraint validation to include the relation making the error message different from previous PG versions.
PG13 adds the relation to constraint violation error messages making the test output different from previous PG versions.
PG13 changes the relation name aliasing in EXPLAIN output making explain output different from previous PG versions.
The order of relations of cluster is different on PG13 compared to previous versions.
499e9f7
to
d769159
Compare
----------------------------------------------------------------------------------------------------------------------------------------------------- | ||
Limit | ||
Output: (time_func(_hyper_3_7_chunk."time")), _hyper_3_7_chunk.device, (avg(_hyper_3_7_chunk.temp)) | ||
-> GroupAggregate |
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.
I see some of the HashAggregate became GroupAggregates (for thsi file) in PG13. Do we need to set enable_hashagg = false here to ensure that plans will not change?
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.
i think the groupagg plans are preferable and i think we dont care what type of aggregation postgres uses here and the test is already pg-version specific so it should be stable
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. Have a question about 1 of the tests.
No description provided.