Re: Implementing Incremental View Maintenance

Hello Zhihong Yu,

I already replied to your comments before, but I forgot to include
the list to CC, so I resend the same again. Sorry for the duplicate
emails.

On Thu, 3 Feb 2022 09:51:52 -0800
Zhihong Yu <zyu(at)yugabyte(dot)com> wrote:

> For CreateIndexOnIMMV():
>
> + ereport(NOTICE,
> + (errmsg("could not create an index on materialized view
> \"%s\" automatically",
> ...
> + return;
> + }
>
> Should the return type be changed to bool so that the caller knows whether
> the index creation succeeds ?
> If index creation is unsuccessful, should the call
> to CreateIvmTriggersOnBaseTables() be skipped ?

CreateIvmTriggersOnBaseTables() have to be called regardless
of whether an index is created successfully or not, so I think
CreateindexOnIMMV() doesn't have to return the result for now.

> For check_ivm_restriction_walker():
>
> + break;
> + expression_tree_walker(node, check_ivm_restriction_walker,
> NULL);
> + break;
>
> Something is missing between the break and expression_tree_walker().

Yes, it's my mistake during making the patch-set. I fixed it in the
updated patch I attached in the other post.

Regards,
Yugo Nagata

--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>