Skip to content
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

Make dist_partial_agg output pg version specific #3682

Merged
merged 1 commit into from Oct 13, 2021

Conversation

svenklemm
Copy link
Member

The plan output of the dist_partial_agg test is different on PG14
so we need to make it PG version specific. On PG14 sorts are pushed
down more often leading to better plans in some cases.

@codecov
Copy link

codecov bot commented Oct 12, 2021

Codecov Report

Merging #3682 (50af911) into master (bfe3603) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3682      +/-   ##
==========================================
- Coverage   90.78%   90.77%   -0.01%     
==========================================
  Files         212      212              
  Lines       36551    36533      -18     
==========================================
- Hits        33182    33163      -19     
- Misses       3369     3370       +1     
Impacted Files Coverage Δ
src/nodes/chunk_dispatch.c 94.44% <ø> (-3.77%) ⬇️
src/process_utility.c 94.44% <ø> (ø)
src/nodes/chunk_insert_state.c 97.62% <100.00%> (-0.12%) ⬇️
src/nodes/hypertable_insert.c 95.67% <100.00%> (-0.03%) ⬇️
tsl/src/remote/dist_ddl.c 97.01% <100.00%> (-0.02%) ⬇️
tsl/src/bgw_policy/job.c 96.85% <0.00%> (-0.01%) ⬇️
tsl/src/nodes/data_node_dispatch.c 97.34% <0.00%> (+0.24%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de2d446...50af911. Read the comment docs.

The plan output of the dist_partial_agg test is different on PG14
so we need to make it PG version specific. On PG14 sorts are pushed
down more often leading to better plans in some cases.
This also updates the dist_hypertable-14 test output which differs
to previous PG versions due to some renumbering of relation aliases.
@svenklemm svenklemm merged commit bfe2b08 into timescale:master Oct 13, 2021
@NunoFilipeSantos NunoFilipeSantos added this to the TimescaleDB 2.5 milestone Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants