From dd03ba9fcaa5d04b28148e2ed6fb533fe1f71433 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 5 Apr 2009 20:32:06 +0000 Subject: [PATCH] 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 markedly cheaper than lappend). However now that EXPLAIN is labeling subplans with IDs that are based on order of creation, this may help produce a slightly less surprising printout. --- src/backend/executor/execQual.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/executor/execQual.c b/src/backend/executor/execQual.c index 8ec92c4ccc..1fac16212a 100644 --- a/src/backend/executor/execQual.c +++ b/src/backend/executor/execQual.c @@ -4326,7 +4326,7 @@ ExecInitExpr(Expr *node, PlanState *parent) sstate = ExecInitSubPlan(subplan, parent); /* Add SubPlanState nodes to parent->subPlan */ - parent->subPlan = lcons(sstate, parent->subPlan); + parent->subPlan = lappend(parent->subPlan, sstate); state = (ExprState *) sstate; } -- 2.39.5