Remove faulty Asserts added by 1999fe227.

I added these assertions in an attempt to show that setting
ret_old_alias/ret_new_alias in set_deparse_plan() fully covered
all the parts of the plan where RETURNING Vars could be found.
But it emerges that the returningOldAlias/returningNewAlias
fields they are filled from can sometimes be NULL themselves,
allowing the asserts to fail.  We discussed various ways that
we might adjust the rewriter to prevent that, but the upshot
of the discussion was just to remove the asserts again.

Reported-by: Ayush Tiwari <[email protected]>
Author: Tom Lane <[email protected]>
Reviewed-by: Dean Rasheed <[email protected]>
Discussion: 
https://postgr.es/m/cajtyswwsorqbyvixmsvt4-nwkjuykkjjk2c+gqtq+vw2pjo...@mail.gmail.com
Backpatch-through: 18

Branch
------
REL_19_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/015f4c9fc35f91a1ab46bafab05a3686cfe09d46

Modified Files
--------------
src/backend/utils/adt/ruleutils.c | 6 ------
1 file changed, 6 deletions(-)

Reply via email to