]> granicus.if.org Git - postgresql/commit
Add some errdetail to checkRuleResultList().
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 2 Jul 2014 16:31:24 +0000 (12:31 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 2 Jul 2014 16:31:24 +0000 (12:31 -0400)
commit7980ab30ecf36162699f138697e2ff5589d9063e
tree9fe0794007b1617a497787e0b6adacc5f030103e
parentd97e98e8337e915e3404f7b2a8a6b966b6f5394f
Add some errdetail to checkRuleResultList().

This function wasn't originally thought to be really user-facing,
because converting a table to a view isn't something we expect people
to do manually.  So not all that much effort was spent on the error
messages; in particular, while the code will complain that you got
the column types wrong it won't say exactly what they are.  But since
we repurposed the code to also check compatibility of rule RETURNING
lists, it's definitely user-facing.  It now seems worthwhile to add
errdetail messages showing exactly what the conflict is when there's
a mismatch of column names or types.  This is prompted by bug #10836
from Matthias Raffelsieper, which might have been forestalled if the
error message had reported the wrong column type as being "record".

Back-patch to 9.4, but not into older branches where the set of
translatable error strings is supposed to be stable.
src/backend/rewrite/rewriteDefine.c