don't modify parent's clauses in create_append_plan_common()

funbringer · funbringer · commit 027fe76c09c2 · 2016-08-31T14:11:28.000+03:00
diff --git a/src/nodes_common.c b/src/nodes_common.c
@@ -372,10 +372,6 @@ create_append_plan_common(PlannerInfo *root, RelOptInfo *rel,
 		{
 			Plan		   *child_plan = (Plan *) lfirst(lc2);
 			RelOptInfo 	   *child_rel = ((Path *) lfirst(lc1))->parent;
-			Oid				child_relid;
-
-			/* Fetch relid of the 'child_rel' */
-			child_relid = root->simple_rte_array[child_rel->relid]->relid;
 
 			/* Replace rel's  tlist with a matching one */
 			if (!cscan->scan.plan.targetlist)
@@ -390,10 +386,6 @@ create_append_plan_common(PlannerInfo *root, RelOptInfo *rel,
 			if (!cscan->custom_scan_tlist)
 				cscan->custom_scan_tlist = replace_tlist_varnos(child_plan->targetlist,
 																rel);
-
-			/* If this is a plan for parent table, fill it with quals */
-			if (PrelParentRelid(prel) == child_relid)
-				child_plan->qual = get_actual_clauses(clauses);
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -372,10 +372,6 @@ create_append_plan_common(PlannerInfo root, RelOptInfo rel,`
`372`	`372`	`{`
`373`	`373`	`Plan child_plan = (Plan ) lfirst(lc2);`
`374`	`374`	`RelOptInfo child_rel = ((Path ) lfirst(lc1))->parent;`
`375`		`- Oid child_relid;`
`376`		`-`
`377`		`- /* Fetch relid of the 'child_rel' */`
`378`		`- child_relid = root->simple_rte_array[child_rel->relid]->relid;`
`379`	`375`
`380`	`376`	`/* Replace rel's tlist with a matching one */`
`381`	`377`	`if (!cscan->scan.plan.targetlist)`
`@@ -390,10 +386,6 @@ create_append_plan_common(PlannerInfo root, RelOptInfo rel,`
`390`	`386`	`if (!cscan->custom_scan_tlist)`
`391`	`387`	`cscan->custom_scan_tlist = replace_tlist_varnos(child_plan->targetlist,`
`392`	`388`	`rel);`
`393`		`-`
`394`		`- /* If this is a plan for parent table, fill it with quals */`
`395`		`- if (PrelParentRelid(prel) == child_relid)`
`396`		`- child_plan->qual = get_actual_clauses(clauses);`
`397`	`389`	`}`
`398`	`390`	`}`
`399`	`391`