diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-03-14 12:16:09 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-03-14 12:16:36 -0400 |
| commit | 0a9d7e1f6d814799e6cd5046513702dd1fe38929 (patch) | |
| tree | 8d7b24ba757fcfe6926b98173dee832477c62062 /src/backend/optimizer/path/allpaths.c | |
| parent | 2455ab48844c90419714e27eafd235a85de23232 (diff) | |
| download | postgresql-0a9d7e1f6d814799e6cd5046513702dd1fe38929.tar.gz postgresql-0a9d7e1f6d814799e6cd5046513702dd1fe38929.zip | |
Ensure dummy paths have correct required_outer if rel is parameterized.
The assertions added by commits 34ea1ab7f et al found another problem:
set_dummy_rel_pathlist and mark_dummy_rel were failing to label
the dummy paths they create with the correct outer_relids, in case
the relation is necessarily parameterized due to having lateral
references in its tlist. It's likely that this has no user-visible
consequences in production builds, at the moment; but still an assertion
failure is a bad thing, so back-patch the fix.
Per bug #15694 from Roman Zharkov (via Alexander Lakhin)
and an independent report by Tushar Ahuja.
Discussion: https://postgr.es/m/15694-74f2ca97e7044f7f@postgresql.org
Discussion: https://postgr.es/m/7d72ab20-c725-3ce2-f99d-4e64dd8a0de6@enterprisedb.com
Diffstat (limited to 'src/backend/optimizer/path/allpaths.c')
| -rw-r--r-- | src/backend/optimizer/path/allpaths.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c index d8ba7add136..b2c5c833f72 100644 --- a/src/backend/optimizer/path/allpaths.c +++ b/src/backend/optimizer/path/allpaths.c @@ -1973,7 +1973,8 @@ set_dummy_rel_pathlist(RelOptInfo *rel) rel->partial_pathlist = NIL; /* Set up the dummy path */ - add_path(rel, (Path *) create_append_path(NULL, rel, NIL, NIL, NULL, + add_path(rel, (Path *) create_append_path(NULL, rel, NIL, NIL, + rel->lateral_relids, 0, false, NIL, -1)); /* |