Re: MERGE SQL statement for PG12

Поиск
Список
Период
Сортировка
От Pavan Deolasee
Тема Re: MERGE SQL statement for PG12
Дата
Msg-id CABOikdPraSx-f8b+Ojp4P_HUVi8pcNVWQu2+VDoX9pnunb5j6w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: MERGE SQL statement for PG12  (Jaime Casanova <jaime.casanova@2ndquadrant.com>)
Ответы Re: MERGE SQL statement for PG12
Список pgsql-hackers


On Sun, Sep 30, 2018 at 2:55 AM Jaime Casanova <jaime.casanova@2ndquadrant.com> wrote:
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);


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
Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Langote
Дата:
Сообщение: Re: CVE-2017-7484-induced bugs, or, btree cmp functions are notleakproof?
Следующее
От: Pavel Stehule
Дата:
Сообщение: Re: PostgreSQL vs SQL/XML Standards