diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-16 18:13:35 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-16 18:13:45 -0400 |
| commit | 9bfd2822b3201f6b0de1e87305b11ee3885b36d9 (patch) | |
| tree | 6a77ebc0607221c4c884c7fa42a14c168179571c /src/backend/optimizer/plan/createplan.c | |
| parent | 0dc40196f27531ab30bbbc516df9894edc276d6e (diff) | |
| download | postgresql-9bfd2822b3201f6b0de1e87305b11ee3885b36d9.tar.gz postgresql-9bfd2822b3201f6b0de1e87305b11ee3885b36d9.zip | |
Enable use of Memoize atop an Append that came from UNION ALL.
create_append_path() would only apply get_baserel_parampathinfo
when the path is for a partitioned table, but it's also potentially
useful for paths for UNION ALL appendrels. Specifically, that
supports building a Memoize path atop this one.
While we're in the vicinity, delete some dead code in
create_merge_append_plan(): there's no need for it to support
parameterized MergeAppend paths, and it doesn't look like that
is going to change anytime soon. It'll be easy enough to undo
this when/if it becomes useful.
Richard Guo
Discussion: https://postgr.es/m/CAMbWs4_ABSu4PWG2rE1q10tJugEXHWgru3U8dAgkoFvgrb6aEA@mail.gmail.com
Diffstat (limited to 'src/backend/optimizer/plan/createplan.c')
| -rw-r--r-- | src/backend/optimizer/plan/createplan.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/src/backend/optimizer/plan/createplan.c b/src/backend/optimizer/plan/createplan.c index fa09a6103b1..910ffbf1e13 100644 --- a/src/backend/optimizer/plan/createplan.c +++ b/src/backend/optimizer/plan/createplan.c @@ -1531,16 +1531,8 @@ create_merge_append_plan(PlannerInfo *root, MergeAppendPath *best_path, prunequal = extract_actual_clauses(rel->baserestrictinfo, false); - if (best_path->path.param_info) - { - List *prmquals = best_path->path.param_info->ppi_clauses; - - prmquals = extract_actual_clauses(prmquals, false); - prmquals = (List *) replace_nestloop_params(root, - (Node *) prmquals); - - prunequal = list_concat(prunequal, prmquals); - } + /* We don't currently generate any parameterized MergeAppend paths */ + Assert(best_path->path.param_info == NULL); if (prunequal != NIL) node->part_prune_index = make_partition_pruneinfo(root, rel, |