]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Remove bogus assertion in transformExpressionList().
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 19 Oct 2021 15:35:15 +0000 (11:35 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 19 Oct 2021 15:35:15 +0000 (11:35 -0400)
commitcf47dc6befe386d16a3f36df6452f07e7910d6b6
tree14cd8136949096eb3fb5dd1d79df500f26ca6dba
parent038892c8101845b6b4364cf2f5773207c4a4525c
Remove bogus assertion in transformExpressionList().

I think when I added this assertion (in commit 8f889b108), 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
src/backend/parser/parse_target.c