database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-29 13:56:47 +03:00
Code

Repair failure to mark an inserted Materialize node with the appropriate

Browse Source 
extParam/locParam lists.  Per bug #526.
This commit is contained in:
Tom Lane 2001-11-30 19:24:15 +00:00
parent 22a6a7e371
commit c31bcbc8d6
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/backend/optimizer/plan/subselect.c
View File

@ -7,7 +7,7 @@
* Portions Copyright (c) 1994, Regents of the University of California * Portions Copyright (c) 1994, Regents of the University of California
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/subselect.c,v 1.49 2001/03/22 03:59:37 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/subselect.c,v 1.50 2001/11/30 19:24:15 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -324,6 +324,12 @@ make_subplan(SubLink *slink)
* is anything more complicated than a plain sequential scan, and * is anything more complicated than a plain sequential scan, and
* we do it even for seqscan if the qual appears selective enough * we do it even for seqscan if the qual appears selective enough
* to eliminate many tuples. * to eliminate many tuples.
*
* XXX It's pretty ugly to be inserting a MATERIAL node at this
* point. Since subquery_planner has already run SS_finalize_plan
* on the subplan tree, we have to kluge up parameter lists for
* the MATERIAL node. Possibly this could be fixed by postponing
* SS_finalize_plan processing until setrefs.c is run.
*/ */
if (node->parParam == NIL) if (node->parParam == NIL)
{ {
@ -362,8 +368,13 @@ make_subplan(SubLink *slink)
} }
if (use_material) if (use_material)
{ {
plan = (Plan *) make_material(plan->targetlist, plan); Plan *matplan;
node->plan = plan;
matplan = (Plan *) make_material(plan->targetlist, plan);
/* kluge --- see comments above */
matplan->extParam = listCopy(plan->extParam);
matplan->locParam = listCopy(plan->locParam);
node->plan = plan = matplan;
} }
} }