Sorry but I found the v7 patch has typo and it can't be built...
I attached fixed one(v8).

2021年4月3日(土) 9:53 Kazutaka Onishi <oni...@heterodb.com>:
>
> All,
>
> Thank you for discussion.
> I've updated the patch (v6->v7) according to the conclusion.
>
> I'll show the modified points:
> 1. Comments for ExecuteTuncate()
> 2. Replacing extra value in frels_extra with integer to label.
> 3. Skipping XLOG_HEAP_TRUNCATE on foreign table
>
> Regards,
>
> 2021年4月2日(金) 11:44 Fujii Masao <masao.fu...@oss.nttdata.com>:
> >
> >
> >
> > On 2021/04/02 9:37, Kohei KaiGai wrote:
> > > It is fair enough for me to reverse the order of actual truncation.
> > >
> > > How about the updated comments below?
> > >
> > >      This is a multi-relation truncate.  We first open and grab exclusive
> > >      lock on all relations involved, checking permissions (local database
> > >      ACLs even if relations are foreign-tables) and otherwise verifying
> > >      that the relation is OK for truncation. In CASCADE mode, ...(snip)...
> > >      Finally all the relations are truncated and reindexed. If any 
> > > foreign-
> > >      tables are involved, its callback shall be invoked prior to the 
> > > truncation
> > >      of regular tables.
> >
> > LGTM.
> >
> >
> > >> BTW, the latest patch doesn't seem to be applied cleanly to the master
> > >> because of commit 27e1f14563. Could you rebase it?
> > >>
> > > Onishi-san, go ahead. :-)
> >
> > +1
> >
> > Regards,
> >
> > --
> > Fujii Masao
> > Advanced Computing Technology Center
> > Research and Development Headquarters
> > NTT DATA CORPORATION

Attachment: pgsql14-truncate-on-foreign-table.v8.patch
Description: Binary data

Reply via email to