From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 12 Jul 2007 18:27:01 +0000 (+0000)
Subject: Fix mistaken Assert in adjust_appendrel_attr_needed, per Greg Stark.
X-Git-Tag: REL8_3_BETA1~468
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bc8d164d066f677b5ef3b9c68e821710b0650023;p=postgresql

Fix mistaken Assert in adjust_appendrel_attr_needed, per Greg Stark.
---

diff --git a/src/backend/optimizer/prep/prepunion.c b/src/backend/optimizer/prep/prepunion.c
index 5e80dc1559..2dc293b74d 100644
--- a/src/backend/optimizer/prep/prepunion.c
+++ b/src/backend/optimizer/prep/prepunion.c
@@ -22,7 +22,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/optimizer/prep/prepunion.c,v 1.142 2007/06/11 01:16:23 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/optimizer/prep/prepunion.c,v 1.143 2007/07/12 18:27:01 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1288,8 +1288,6 @@ adjust_appendrel_attr_needed(RelOptInfo *oldrel, AppendRelInfo *appinfo,
 	ListCell   *lm;
 
 	/* Create empty result array */
-	Assert(new_min_attr <= oldrel->min_attr);
-	Assert(new_max_attr >= oldrel->max_attr);
 	new_attr_needed = (Relids *)
 		palloc0((new_max_attr - new_min_attr + 1) * sizeof(Relids));
 	/* Process user attributes, with appropriate attno mapping */
@@ -1312,6 +1310,7 @@ adjust_appendrel_attr_needed(RelOptInfo *oldrel, AppendRelInfo *appinfo,
 		parent_attr++;
 	}
 	/* Process system attributes, including whole-row references */
+	Assert(new_min_attr <= oldrel->min_attr);
 	for (parent_attr = oldrel->min_attr; parent_attr <= 0; parent_attr++)
 	{
 		Relids		attrneeded;