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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers