aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/nodeResult.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2007-02-16 03:49:04 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2007-02-16 03:49:04 +0000
commit7ea758b0b1e51fd5796f3d599b144e796586d6e8 (patch)
tree5d50ce7920172d6d3b0d860f0bf6ae8a51875f2a /src/backend/executor/nodeResult.c
parent4ebb0cf9c30c1e477d5e2dfcc1f2c016c3f8bbcf (diff)
downloadpostgresql-7ea758b0b1e51fd5796f3d599b144e796586d6e8.tar.gz
postgresql-7ea758b0b1e51fd5796f3d599b144e796586d6e8.zip
Fix another problem in 8.2 changes that allowed "one-time" qual conditions to
be checked at plan levels below the top; namely, we have to allow for Result nodes inserted just above a nestloop inner indexscan. Should think about using the general Param mechanism to pass down outer-relation variables, but for the moment we need a back-patchable solution. Per report from Phil Frost.
Diffstat (limited to 'src/backend/executor/nodeResult.c')
-rw-r--r--src/backend/executor/nodeResult.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/src/backend/executor/nodeResult.c b/src/backend/executor/nodeResult.c
index 2d20079569f..98e9d219727 100644
--- a/src/backend/executor/nodeResult.c
+++ b/src/backend/executor/nodeResult.c
@@ -38,7 +38,7 @@
* Portions Copyright (c) 1994, Regents of the University of California
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/executor/nodeResult.c,v 1.38 2007/02/15 03:07:13 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/executor/nodeResult.c,v 1.39 2007/02/16 03:49:04 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@@ -309,10 +309,12 @@ ExecReScanResult(ResultState *node, ExprContext *exprCtxt)
node->rs_checkqual = (node->resconstantqual == NULL) ? false : true;
/*
- * if chgParam of subnode is not null then plan will be re-scanned by
- * first ExecProcNode.
+ * If chgParam of subnode is not null then plan will be re-scanned by
+ * first ExecProcNode. However, if caller is passing us an exprCtxt
+ * then forcibly rescan the subnode now, so that we can pass the
+ * exprCtxt down to the subnode (needed for gated indexscan).
*/
- if (((PlanState *) node)->lefttree &&
- ((PlanState *) node)->lefttree->chgParam == NULL)
- ExecReScan(((PlanState *) node)->lefttree, exprCtxt);
+ if (node->ps.lefttree &&
+ (node->ps.lefttree->chgParam == NULL || exprCtxt != NULL))
+ ExecReScan(node->ps.lefttree, exprCtxt);
}