From: | Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> |
---|---|
To: | nagata(at)sraoss(dot)co(dot)jp |
Cc: | pgsql-hackers(at)postgresql(dot)org, takuma(dot)hoshiai(at)gmail(dot)com, michael(at)paquier(dot)xyz, amitlangote09(at)gmail(dot)com, alvherre(at)2ndquadrant(dot)com, thomas(dot)munro(at)gmail(dot)com, kgrittn(at)gmail(dot)com |
Subject: | Re: Implementing Incremental View Maintenance |
Date: | 2020-08-21 12:40:50 |
Message-ID: | 20200821.214050.1511052602383222423.t-ishii@sraoss.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
From: Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
Subject: Re: Implementing Incremental View Maintenance
Date: Fri, 21 Aug 2020 17:23:20 +0900
Message-ID: <20200821172320(dot)a2506577d5244b6066f69331(at)sraoss(dot)co(dot)jp>
> On Wed, 19 Aug 2020 10:02:42 +0900 (JST)
> Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Sharma | 2020-08-21 13:24:58 | Re: recovering from "found xmin ... from before relfrozenxid ..." |
Previous Message | Andrey Lepikhov | 2020-08-21 12:19:06 | Re: [POC] Fast COPY FROM command for the table with foreign partitions |