Hello,

On Thu, Jun 15, 2023 at 5:07 PM Sho Kato (Fujitsu) <kato-...@fujitsu.com> wrote:
>
> Hi,
>
> I've attached the patch for the following rewriteTargetView comments.
>
>
>   Assert(parsetree->resultRelation == new_rt_index);
>
>     /*
>      * For INSERT/UPDATE we must also update resnos in the targetlist to refer
>      * to columns of the base relation, since those indicate the target
>      * columns to be affected.
>      *
>      * Note that this destroys the resno ordering of the targetlist, but that
>      * will be fixed when we recurse through rewriteQuery, which will invoke
>      * rewriteTargetListIU again on the updated targetlist.
>      */
>     if (parsetree->commandType != CMD_DELETE)
>     {
>         foreach(lc, parsetree->targetList)
>
> s/rewriteQuery/RewriteQuery

Good catch and thanks for the patch.  Will push shortly.

-- 
Thanks, Amit Langote
EDB: http://www.enterprisedb.com


Reply via email to