Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit4ebca55

Browse files
committed
Fix generate_partitionwise_join_paths() to tolerate failure.
We might fail to generate a partitionwise join, becausereparameterize_path_by_child() does not support all path types.This should not be a hard failure condition: we should just fall backto a non-partitioned join. However, generate_partitionwise_join_pathsdid not consider this possibility and would emit the (misleading)error "could not devise a query plan for the given query" if we'dfailed to make any paths for a child join. Fix it to give up onpartitionwise joining if so. (The accepted technique for giving upappears to be to set rel->nparts = 0, which I find pretty bizarre,but there you have it.)I have not added a test case because there'd be little point:any omissions of this sort that we identify would soon get fixedby extending reparameterize_path_by_child(), so the test would stopproving anything. However, right now there is a known test case basedon failure to cover MaterialPath, and with that I've found that thisis broken in all supported versions. Hence, patch all the way back.Original report and patch by me; thanks to Richard Guo foridentifying a test case that works against committed versions.Discussion:https://postgr.es/m/1854233.1669949723@sss.pgh.pa.us
1 parent3bed881 commit4ebca55

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

‎src/backend/optimizer/path/allpaths.c

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3943,12 +3943,24 @@ generate_partitionwise_join_paths(PlannerInfo *root, RelOptInfo *rel)
39433943
if (child_rel==NULL)
39443944
continue;
39453945

3946-
/*Add partitionwise join paths for partitioned child-joins. */
3946+
/*Make partitionwise join paths forthispartitioned child-join. */
39473947
generate_partitionwise_join_paths(root,child_rel);
39483948

3949+
/* If we failed to make any path for this child, we must give up. */
3950+
if (child_rel->pathlist==NIL)
3951+
{
3952+
/*
3953+
* Mark the parent joinrel as unpartitioned so that later
3954+
* functions treat it correctly.
3955+
*/
3956+
rel->nparts=0;
3957+
return;
3958+
}
3959+
3960+
/* Else, identify the cheapest path for it. */
39493961
set_cheapest(child_rel);
39503962

3951-
/* Dummy childrenwill not be scanned, so ignore those. */
3963+
/* Dummy childrenneed not be scanned, so ignore those. */
39523964
if (IS_DUMMY_REL(child_rel))
39533965
continue;
39543966

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp