Re: MERGE SQL statement for PG12

On Sun, Sep 30, 2018 at 2:55 AM Jaime Casanova <
jaime(dot)casanova(at)2ndquadrant(dot)com> wrote:

> On Mon, 24 Sep 2018 at 05:15, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>
> wrote:
> >
> > A new version rebased against the current master is attached.
> >
>
> Hi Pavan,
>
> A day after you posted this patch commit
> 29c94e03c7d05d2b29afa1de32795ce178531246 removed ExecStoreTuple.
> I'm right in believe that the change in
> src/backend/executor/execMerge.c should be for ExecStoreHeapTuples?
>
> - ExecStoreTuple(&tuple, mtstate->mt_existing, buffer, false);
> + ExecStoreHeapTuple(&tuple, mtstate->mt_existing, false);
>
>
Hi Jaime,

Thanks for keeping an eye on the patch. I've rebased the patch against the
current master. A new version is attached.

Thanks,
Pavan

--
Pavan Deolasee http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services