Re: Update comments in nodeModifyTable.c

On 2017/08/02 4:07, Robert Haas wrote:
> On Tue, Aug 1, 2017 at 12:31 AM, Etsuro Fujita
> <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)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