Commit5da466c

committed

Make sure targetlist generated for subplan does not share

nodes with HAVING qualifier of upper plan. Have not seen any failures,just being a little bit paranoid...

1 parentb6c732e commit5da466cCopy full SHA for 5da466c

File tree

-2

lines changed

-2

lines changed

Lines changed: 7 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`*`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.48 1999/05/03 00:38:43 tgl Exp $`
	`10`	`+ * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.49 1999/05/04 00:00:20 tgl Exp $`
`11`	`11`	`*`
`12`	`12`	`*-------------------------------------------------------------------------`
`13`	`13`	`*/`
`@@ -505,8 +505,13 @@ make_subplanTargetList(Query *parse,`
`505`	`505`
`506`	`506`	`if (tlist_member(v,sub_tlist)==NULL)`
`507`	`507`	`{`
	`508`	`+/* Make sure sub_tlist element is a fresh object not shared with`
	`509`	`+ * any other structure; not sure if anything will break if it is`
	`510`	`+ * shared, but better to be safe...`
	`511`	`+ */`
`508`	`512`	`sub_tlist=lappend(sub_tlist,`
`509`		`-create_tl_element(v,next_resno));`
	`513`	`+create_tl_element((Var*)copyObject(v),`
	`514`	`+next_resno));`
`510`	`515`	`next_resno++;`
`511`	`516`	`}`
`512`	`517`	`}`

Comments

(0)