Hi,
I updated the wiki page.
https://wiki.postgresql.org/wiki/Incremental_View_Maintenance
On Fri, 21 Aug 2020 21:40:50 +0900 (JST)
Tatsuo Ishii <ishii@sraoss.co.jp> wrote:
> From: Yugo NAGATA <nagata@sraoss.co.jp>
> Subject: Re: Implementing Incremental View Maintenance
> Date: Fri, 21 Aug 2020 17:23:20 +0900
> Message-ID: <20200821172320.a2506577d5244b6066f69331@sraoss.co.jp>
>
> > On Wed, 19 Aug 2020 10:02:42 +0900 (JST)
> > Tatsuo Ishii <ishii@sraoss.co.jp> wrote:
> >
> >> I have looked into this.
> >
> > Thank you for your reviewing!
> >
> >> - 0004-Allow-to-prolong-life-span-of-transition-tables-unti.patch:
> >> This one needs a comment to describe what the function does etc.
> >>
> >> +void
> >> +SetTransitionTablePreserved(Oid relid, CmdType cmdType)
> >> +{
> >
> > I added a comment for this function and related places.
> >
> > +/*
> > + * SetTransitionTablePreserved
> > + *
> > + * Prolong lifespan of transition tables corresponding specified relid and
> > + * command type to the end of the outmost query instead of each nested query.
> > + * This enables to use nested AFTER trigger's transition tables from outer
> > + * query's triggers. Currently, only immediate incremental view maintenance
> > + * uses this.
> > + */
> > +void
> > +SetTransitionTablePreserved(Oid relid, CmdType cmdType)
> >
> > Also, I removed releted unnecessary code which was left accidentally.
> >
> >
> >> - 0007-Add-aggregates-support-in-IVM.patch
> >> "Check if the given aggregate function is supporting" shouldn't be
> >> "Check if the given aggregate function is supporting IVM"?
> >
> > Yes, you are right. I fixed this, too.
> >
> >>
> >> + * check_aggregate_supports_ivm
> >> + *
> >> + * Check if the given aggregate function is supporting
>
> Thanks for the fixes. I have changed the commit fest status to "Ready
> for Committer".
>
> Best regards,
> --
> Tatsuo Ishii
> SRA OSS, Inc. Japan
> English: http://www.sraoss.co.jp/index_en.php
> Japanese:http://www.sraoss.co.jp
>
>
--
Yugo NAGATA <nagata@sraoss.co.jp>