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