(2019/04/26 13:58), Etsuro Fujita wrote:
(2019/04/26 13:20), Amit Langote wrote:
+ Note that this function is also called when inserting routed tuples
into
+ a foreign-table partition or executing<command>COPY FROM</command> on
+ a foreign table, in which case it is called in a different way than it
+ is in the<command>INSERT</command> case.

Maybe minor, but should the last part of this sentence read as:

...in which case it is called in a different way than it is in the case
<command>INSERT</command> is operating directly on the foreign table.

?

Yeah, but I think it would be OK to just say "the INSERT case" because
this note is added to the docs for ExecForeignInsert(), which allows the
FDW to directly insert into foreign tables as you know, so users will
read "the INSERT case" as "the case <command>INSERT</command> is
operating directly on the foreign table".

Pushed as-is.  I think we can change that later if necessary.

Best regards,
Etsuro Fujita



Reply via email to