Re: partition tree inspection functions

Hi Amit,

On 07/26/2018 10:33 PM, Amit Langote wrote:
> Optional parameter sounds good, so made it get_partition_level(regclass [
> , regclass ]) in the updated patch.  Although, adding that argument is not
> without possible surprises its result might evoke.  Like, what happens if
> you try to find the level of the root table by passing a leaf partition
> oid for the root table argument, or pass a totally unrelated table for the
> root table argument.  For now, I've made the function return 0 for such cases.
> 

As 0 is a valid return value for root nodes I think we should use -1 
instead for these cases.

Otherwise looks good.

Best regards,
  Jesper