Re: Constraint trigger doc patch

Manual page changed applied.  Thanks.

---------------------------------------------------------------------------


Michael Paesold wrote:
> [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
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: if posting/reading through Usenet, please send an appropriate
>        subscribe-nomail command to majordomo@postgresql.org so that your
>        message can get through to the mailing list cleanly

--
  Bruce Momjian   bruce@momjian.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +