Re: MERGE SQL statement for PG12 - Mailing list pgsql-hackers

From Jaime Casanova
Subject Re: MERGE SQL statement for PG12
Date
Msg-id CAJGNTeP3NGjA3iWUZY2WtX9N-vDg0H589yyXqM_j0CcWzUsjEg@mail.gmail.com
Whole thread Raw
In response to Re: MERGE SQL statement for PG12  (Pavan Deolasee <pavan.deolasee@gmail.com>)
Responses Re: MERGE SQL statement for PG12
List pgsql-hackers
On Mon, 24 Sep 2018 at 05:15, Pavan Deolasee <pavan.deolasee@gmail.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);


-- 
Jaime Casanova                      www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Cygwin linking rules
Next
From: Sergei Kornilov
Date:
Subject: Re: Continue work on changes to recovery.conf API