pgsql: Improve error message for rejecting RETURNING clauses with dropp - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Improve error message for rejecting RETURNING clauses with dropp
Date
Msg-id E1aaXvh-0003YP-Ui@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Improve error message for rejecting RETURNING clauses with dropped columns.

This error message was written with only ON SELECT rules in mind, but since
then we also made RETURNING-clause targetlists go through the same logic.
This means that you got a rather off-topic error message if you tried to
add a rule with RETURNING to a table having dropped columns.  Ideally we'd
just support that, but some preliminary investigation says that it might be
a significant amount of work.  Seeing that Nicklas Avén's complaint is the
first one we've gotten about this in the ten years or so that the code's
been like that, I'm unwilling to put much time into it.  Instead, improve
the error report by issuing a different message for RETURNING cases, and
revise the associated comment based on this investigation.

Discussion: 1456176604.17219.9.camel@jordogskog.no

Branch
------
REL9_5_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/ca778d01e7f0f216bf2a8b0f45952c4c3ef5321a

Modified Files
--------------
src/backend/rewrite/rewriteDefine.c | 26 +++++++++++++++++++-------
1 file changed, 19 insertions(+), 7 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Minor tweaks for new src/test/recovery
Next
From: Tom Lane
Date:
Subject: pgsql: Improve error message for rejecting RETURNING clauses with dropp