From: Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Cc: Rushabh Lathia <rushabh.lathia@gmail.com>
Cc: Michael Paquier <michael.paquier@gmail.com>
Cc: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
Subject: Re: postgres_fdw: perform UPDATE/DELETE .. RETURNING on a join directly
Date: Thu, 08 Feb 2018 14:53:38 +0900
Message-ID: <5A7BE5E2.6040306@lab.ntt.co.jp> (raw)
In-Reply-To: <CA+TgmoYef9czAA1gJeqh4v47Tv=dqOp4EfdU6bV8EYkOdCwvUw@mail.gmail.com>
References: <5A438A45.2000909@lab.ntt.co.jp>
<5A57193A.2080003@lab.ntt.co.jp>
<CA+TgmoYef9czAA1gJeqh4v47Tv=dqOp4EfdU6bV8EYkOdCwvUw@mail.gmail.com>
(2018/02/08 5:39), Robert Haas wrote:
> On Thu, Jan 11, 2018 at 2:58 AM, Etsuro Fujita
> <fujita.etsuro@lab.ntt.co.jp> wrote:
>> (2017/12/27 20:55), Etsuro Fujita wrote:
>>> Attached is an updated version of the patch.
>>
>> I revised code/comments a little bit. PFA new version.
>
> I spent a while reading through this today. I see a few decisions
> here or there that are debatable, in the sense that somebody else
> might have chosen to do it differently, but I don't see anything that
> actually looks wrong. So, committed.
Thanks for committing, Robert! Thanks for reviewing, Rushabh and Ashutosh!
Best regards,
Etsuro Fujita
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: fujita.etsuro@lab.ntt.co.jp, robertmhaas@gmail.com, rushabh.lathia@gmail.com, michael.paquier@gmail.com, ashutosh.bapat@enterprisedb.com
Subject: Re: postgres_fdw: perform UPDATE/DELETE .. RETURNING on a join directly
In-Reply-To: <5A7BE5E2.6040306@lab.ntt.co.jp>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox