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_18_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/b049b4e7b4a159d373ef394bda67f7cf25758998 Modified Files -------------- src/backend/utils/adt/ruleutils.c | 6 ------ 1 file changed, 6 deletions(-)
