NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commiteb4c723

committed

Make ExecInitExpr build the list of SubPlans found in a plan tree in order

of discovery, rather than reverse order. This doesn't matter functionally(I suppose the previous coding dates from the time when lcons was markedlycheaper than lappend). However now that EXPLAIN is labeling subplans withIDs that are based on order of creation, this may help produce a slightlyless surprising printout.

1 parentfbcce08 commiteb4c723Copy full SHA for eb4c723

File tree

1 file changed

-2

lines changed

src/backend/executor
- execQual.c

1 file changed

-2

lines changed

`‎src/backend/executor/execQual.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/executor/execQual.c,v 1.244 2009/04/02 22:39:29 tgl Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/executor/execQual.c,v 1.245 2009/04/05 20:32:06 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -4326,7 +4326,7 @@ ExecInitExpr(Expr node, PlanState parent)`
`4326`	`4326`	`sstate=ExecInitSubPlan(subplan,parent);`
`4327`	`4327`
`4328`	`4328`	`/* Add SubPlanState nodes to parent->subPlan */`
`4329`		`-parent->subPlan=lcons(sstate,parent->subPlan);`
	`4329`	`+parent->subPlan=lappend(parent->subPlan,sstate);`
`4330`	`4330`
`4331`	`4331`	`state= (ExprState*)sstate;`
`4332`	`4332`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commiteb4c723

File tree

1 file changed

1 file changed

`‎src/backend/executor/execQual.c`

0 commit comments