On 2019/03/22 13:12, Michael Paquier wrote:
> On Fri, Mar 22, 2019 at 01:09:23PM +0900, Amit Langote wrote:
>> /* Fetch the top-most parent */
>> ancestors = get_partition_ancestors(relid);
>>
>> Maybe, the patch should update this comment to say "Fetch the list of
>> ancestors".
>
> Perhaps so. Note that actually the list is fetched to grab the
> top-most parent from it, so it sort of makes sense?
It looked fine before, but the new lines added by patch makes it look
wrong/misplaced somehow.
Thanks,
Amit