From: | Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp> |
---|---|
To: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Support TRUNCATE triggers on foreign tables |
Date: | 2022-07-08 02:19:59 |
Message-ID: | 20220708111959.6681569f06347d5f4f9ef32e@sraoss.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello Fujii-san,
Thank you for reviewing the patch!
On Fri, 8 Jul 2022 00:54:37 +0900
Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote:
>
>
> On 2022/06/30 19:38, Yugo NAGATA wrote:
> > Hello,
> >
> > I propose supporting TRUNCATE triggers on foreign tables
> > because some FDW now supports TRUNCATE. I think such triggers
> > are useful for audit logging or for preventing undesired
> > truncate.
> >
> > Patch attached.
>
> Thanks for the patch! It looks good to me except the following thing.
>
> <entry align="center"><command>TRUNCATE</command></entry>
> <entry align="center">—</entry>
> - <entry align="center">Tables</entry>
> + <entry align="center">Tables and foreign tables</entry>
> </row>
>
> You added "foreign tables" for BEFORE statement-level trigger as the above, but ISTM that you also needs to do that for AFTER statement-level trigger. No?
Oops, I forgot it. I attached the updated patch.
Regards,
Yugo Nagata
>
> Regards,
>
> --
> Fujii Masao
> Advanced Computing Technology Center
> Research and Development Headquarters
> NTT DATA CORPORATION
--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
Attachment | Content-Type | Size |
---|---|---|
v2_truncate_trigger_on_fdw.patch | text/x-diff | 4.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Smith | 2022-07-08 02:26:28 | Re: Support logical replication of DDLs |
Previous Message | Ian Barwick | 2022-07-08 02:12:28 | Re: Fast COPY FROM based on batch insert |