From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 9 Nov 2000 18:12:53 +0000 (+0000)
Subject: ExecEndAppend() neglected to close indices on appended result rels,
X-Git-Tag: REL7_1_BETA~216
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=343f615e3ada8a7e9662b33341b73f09e182d105;p=postgresql

ExecEndAppend() neglected to close indices on appended result rels,
and improperly prevented the main result rel from being closed if it
wasn't one of the Append's own result rels.  Per report from Hiroshi.
---

diff --git a/src/backend/executor/nodeAppend.c b/src/backend/executor/nodeAppend.c
index 6c547854b5..81eeb1e8b0 100644
--- a/src/backend/executor/nodeAppend.c
+++ b/src/backend/executor/nodeAppend.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/executor/nodeAppend.c,v 1.36 2000/10/05 19:11:26 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/executor/nodeAppend.c,v 1.37 2000/11/09 18:12:53 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -493,27 +493,32 @@ ExecEndAppend(Append *node)
 
 	/* ----------------
 	 *	close out the different result relations
+	 *
+	 *	NB: this must agree with what EndPlan() does to close a result rel
 	 * ----------------
 	 */
 	resultRelationInfoList = appendstate->as_result_relation_info_list;
 	while (resultRelationInfoList != NIL)
 	{
 		RelationInfo *resultRelationInfo;
-		Relation	resultRelationDesc;
 
 		resultRelationInfo = (RelationInfo *) lfirst(resultRelationInfoList);
-		resultRelationDesc = resultRelationInfo->ri_RelationDesc;
-		heap_close(resultRelationDesc, NoLock);
+
+		heap_close(resultRelationInfo->ri_RelationDesc, NoLock);
+		/* close indices on the result relation, too */
+		ExecCloseIndices(resultRelationInfo);
+
+		/*
+		 * estate may (or may not) be pointing at one of my result relations.
+		 * If so, make sure EndPlan() doesn't try to close it again!
+		 */
+		if (estate->es_result_relation_info == resultRelationInfo)
+			estate->es_result_relation_info = NULL;
+
 		pfree(resultRelationInfo);
 		resultRelationInfoList = lnext(resultRelationInfoList);
 	}
 	appendstate->as_result_relation_info_list = NIL;
-	/*
-	 * This next step is critical to prevent EndPlan() from trying to close
-	 * an already-closed-and-deleted RelationInfo --- es_result_relation_info
-	 * is pointing at one of the nodes we just zapped above.
-	 */
-	estate->es_result_relation_info = NULL;
 
 	/*
 	 * XXX should free appendstate->as_junkfilter_list here