On 2017/06/22 11:25, Etsuro Fujita wrote:
> On 2017/06/21 23:52, Robert Haas wrote:
>
>> You're right that there is a comment missing from the function header,
>> but it's not too hard to figure out what it should be. Just consult
>> the definition of ModifyTable itself:
>>
>> /* RT indexes of non-leaf tables in a partition tree */
>> List *partitioned_rels;
>
> I agree on that point, but I think it's better to add the missing comment
> for the create_modifytable_path argument, as proposed in [1].
Thanks for sharing that link. I was about to send a patch to add the
comment, but seems like you beat me to it.
Thanks,
Amit