Re: Constraint trigger doc patch

[Added pgsql-hackers to CC:]

Michael Glaesemann wrote:
[ a patch for constraint trigger docs]

Great! I was just going to try to use constraint triggers (because I
needed deferrable constraints). So I personally appreciate this
documentation update very much.

Just some notes:

> Index: doc/src/sgml/ref/create_constraint.sgml
...
> --- 21,32 ----
> <refsynopsisdiv>
> <synopsis>
> CREATE CONSTRAINT TRIGGER <replaceable
> class="parameter">name</replaceable>
> ! AFTER <replaceable class="parameter">event [ OR ... ]</replaceable>
> ! ON <replaceable class="parameter">table_name</replaceable>
> ! [ FROM <replaceable
> class="parameter">referenced_table_name</replaceable> ]
> ! { NOT DEFERRABLE | [ DEFERABBLE ] { INITIALLY IMMEDIATE |
> INITIALLY DEFERRED } }
> ! FOR EACH ROW
> ! EXECUTE PROCEDURE <replaceable
> class="parameter">funcname</replaceable> ( <replaceable
> class="parameter">arguments</replaceable> )
> </synopsis>
> </refsynopsisdiv>

It's spelled DEFERRABLE. You got it right in NOT DEFERRABLE but wrong in
[ DEFERABBLE ].

> The name of the constraint trigger. The actual name of the
> created trigger will be of the form
> <literal>RI_ConstraintTrigger_0000<literal> (where 0000 is some number
> assigned by the server). Use this assigned name is when dropping the
> constraint.

It think you should drop the "is" from the last sentence here.
Additionally, I would prefer "Use this assigned name when dropping the
trigger." here, because this one confused me to try to "ALTER TABLE DROP
CONSTRAINT" instead of "DROP TRIGGER".

Thanks again.

Best Regards,
Michael Paesold