Remove bogus assertion in transformExpressionList().

tglsfdc · tglsfdc · commit cf47dc6befe3 · 2021-10-19T11:35:15.000-04:00
I think when I added this assertion (in commit 8f889b1), I was only thinking of the use of transformExpressionList at top level of INSERT and VALUES. But it's also called by transformRowExpr(), which can certainly occur in an UPDATE targetlist, so it's inappropriate to suppose that p_multiassign_exprs must be empty. Besides, since the input is not expected to contain ResTargets, there's no reason it should contain MultiAssignRefs either. Hence this code need not be concerned about the state of p_multiassign_exprs, and we should just drop the assertion. Per bug #17236 from ocean_li_996. It's been wrong for years, so back-patch to all supported branches. Discussion: https://postgr.es/m/17236-3210de9bcba1d7ca@postgresql.org
diff --git a/src/backend/parser/parse_target.c b/src/backend/parser/parse_target.c
@@ -219,7 +219,9 @@ transformTargetList(ParseState *pstate, List *targetlist,
  * This is the identical transformation to transformTargetList, except that
  * the input list elements are bare expressions without ResTarget decoration,
  * and the output elements are likewise just expressions without TargetEntry
- * decoration.  We use this for ROW() and VALUES() constructs.
+ * decoration.  Also, we don't expect any multiassign constructs within the
+ * list, so there's nothing to do for that.  We use this for ROW() and
+ * VALUES() constructs.
  *
  * exprKind is not enough to tell us whether to allow SetToDefault, so
  * an additional flag is needed for that.
@@ -281,9 +283,6 @@ transformExpressionList(ParseState *pstate, List *exprlist,
 		result = lappend(result, e);
 	}
 
-	/* Shouldn't have any multiassign items here */
-	Assert(pstate->p_multiassign_exprs == NIL);
-
 	return result;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,9 @@ transformTargetList(ParseState pstate, List targetlist,`
`219`	`219`	`* This is the identical transformation to transformTargetList, except that`
`220`	`220`	`* the input list elements are bare expressions without ResTarget decoration,`
`221`	`221`	`* and the output elements are likewise just expressions without TargetEntry`
`222`		`- * decoration. We use this for ROW() and VALUES() constructs.`
	`222`	`+ * decoration. Also, we don't expect any multiassign constructs within the`
	`223`	`+ * list, so there's nothing to do for that. We use this for ROW() and`
	`224`	`+ * VALUES() constructs.`
`223`	`225`	`*`
`224`	`226`	`* exprKind is not enough to tell us whether to allow SetToDefault, so`
`225`	`227`	`* an additional flag is needed for that.`
`@@ -281,9 +283,6 @@ transformExpressionList(ParseState pstate, List exprlist,`
`281`	`283`	`result = lappend(result, e);`
`282`	`284`	`}`
`283`	`285`
`284`		`- /* Shouldn't have any multiassign items here */`
`285`		`- Assert(pstate->p_multiassign_exprs == NIL);`
`286`		`-`
`287`	`286`	`return result;`
`288`	`287`	`}`
`289`	`288`