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
