Itagaki-san, I don't have any more comments in this patch, so I hope it to be reviewed by committers then upstreamed.
Thanks for your good jobs. Itagaki Takahiro wrote: > KaiGai Kohei <kai...@ak.jp.nec.com> wrote: > >> In addition, I could find a few matters. >> * TOAST may be necessary for pg_trigger? > > I added toast relation to pg_trigger. > DECLARE_TOAST(pg_trigger, 2336, 2337); > > I think having a toast relation for pg_trigger is reasonable > because pg_trigger already has a variable field "tgargs" > even if we don't have the new field "tgqual" from the patch. > I'm not sure why we don't have a toast relation for pg_trigger > because user might pass very long trigger arguments. > >> * ROW INSERT TRIGGER on COPY FROM statement > > Thanks. Good catch! Fixed and regression test added. > >> * Using system column in WHEN clause >> 2) Describe a notice on the user documentation not to use system columns >> in the WHEN clause, because these are assigned on after the trigger >> invocations. > > I'd like to only add documentation because I don't have a whole solution. > ---- > System columns are not available in the <literal>WHEN</> clause > because those values are initialized after triggers are called. > They might return wrong values if they used in expressions of the clause. > ---- > > Regards, > --- > ITAGAKI Takahiro > NTT Open Source Software Center > -- OSS Platform Development Division, NEC KaiGai Kohei <kai...@ak.jp.nec.com> -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers