Fix CreatePolicy, pg_dump -v; psql and doc updates
authorStephen Frost <sfrost@snowman.net>
Fri, 3 Oct 2014 20:31:53 +0000 (16:31 -0400)
committerStephen Frost <sfrost@snowman.net>
Fri, 3 Oct 2014 20:31:53 +0000 (16:31 -0400)
commit78d72563ef141ddc507ddd5ae77db613a309041a
tree0961d673236ce0e58a76d59b79aff7e1b5235fed
parent596857043023738099d6d16f8edbe6b7353876c0
Fix CreatePolicy, pg_dump -v; psql and doc updates

Peter G pointed out that valgrind was, rightfully, complaining about
CreatePolicy() ending up copying beyond the end of the parsed policy
name.  Name is a fixed-size type and we need to use namein (through
DirectFunctionCall1()) to flush out the entire array before we pass
it down to heap_form_tuple.

Michael Paquier pointed out that pg_dump --verbose was missing a
newline and Fabrízio de Royes Mello further pointed out that the
schema was also missing from the messages, so fix those also.

Also, based on an off-list comment from Kevin, rework the psql \d
output to facilitate copy/pasting into a new CREATE or ALTER POLICY
command.

Lastly, improve the pg_policies view and update the documentation for
it, along with a few other minor doc corrections based on an off-list
discussion with Adam Brightwell.
doc/src/sgml/catalogs.sgml
src/backend/catalog/system_views.sql
src/backend/commands/policy.c
src/bin/pg_dump/pg_dump.c
src/bin/psql/describe.c
src/test/regress/expected/rules.out