Re: Confusing docs about GetForeignUpperPaths in fdwhandler.sgml

From: Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>
To: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Confusing docs about GetForeignUpperPaths in fdwhandler.sgml
Date: 2016-08-31 09:01:41
Message-ID: 2034d93c-9b5a-4c5a-8825-d154e7328e4e@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2016/08/01 21:14, Etsuro Fujita wrote:
> I noticed that the following note about direct modification via
> GetForeignUpperPaths in fdwhandler.sgml is a bit confusing. We have
> another approach using PlanDirectModify, so that should be reflected in
> the note as well. Please find attached a patch.
>
> <function>PlanForeignModify</> and the other callbacks described in
> <xref linkend="fdw-callbacks-update"> are designed around the
> assumption
> that the foreign relation will be scanned in the usual way and then
> individual row updates will be driven by a local
> <literal>ModifyTable</>
> plan node. This approach is necessary for the general case where an
> update requires reading local tables as well as foreign tables.
> However, if the operation could be executed entirely by the foreign
> server, the FDW could generate a path representing that and insert it
> into the <literal>UPPERREL_FINAL</> upper relation, where it would
> compete against the <literal>ModifyTable</> approach.

I'll add this to the upcoming CF.

Best regards,
Etsuro Fujita

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2016-08-31 10:39:29 Re: GIN logging GIN_SEGMENT_UNMODIFIED actions?
Previous Message Aleksander Alekseev 2016-08-31 08:40:25 Re: Small patch for snapmgr.c