On 2017/08/02 4:07, Robert Haas wrote:
> On Tue, Aug 1, 2017 at 12:31 AM, Etsuro Fujita
> <fujita.etsuro@lab.ntt.co.jp> wrote:
>> Maybe I'm missing something, but I'm not sure that's a good idea because the
>> change says like we might have 'wholerow' only for the FDW case, but that
>> isn't correct because we would have 'wholerow' for a view as well. ISTM that
>> views should be one of the typical cases, so I'd like to propose to modify
>> the sentence starting with 'Typically' to something like this: "Typically,
>> this will be a 'ctid' or 'wholerow' attribute, but in the case of a foreign
>> data wrapper it might be a set of junk attributes sufficient to identify the
>> remote row." What do you think about that?
>
> Works for me.
I updated the patch that way. Attached is a new version of the patch.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers