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? > What does "listed partition" mean? Maybe, you mean "If the input > relation..."? How about write this as: Input relation sounds better. -- Michael
signature.asc
Description: PGP signature
