Re: pg_partition_tree crashes for a non-defined relation - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: pg_partition_tree crashes for a non-defined relation
Date
Msg-id 20181210132143.GJ3415@tamriel.snowman.net
Whole thread Raw
In response to Re: pg_partition_tree crashes for a non-defined relation  (Michael Paquier <michael@paquier.xyz>)
Responses Re: pg_partition_tree crashes for a non-defined relation
List pgsql-hackers
Greetings,

* Michael Paquier (michael@paquier.xyz) wrote:
> On Sun, Dec 09, 2018 at 12:49:18PM -0500, Stephen Frost wrote:
> > Looks alright on a quick glance, but shouldn't you also update the
> > comment..?
>
> The comment on HEAD or with the patch is that:
>     /* Only allow relation types that can appear in partition trees. */
>
> This still looks adapted to me.  Or would you reword it because "allow"
> rather implies that an ERROR is returned?  Would you prefer changing it
> something like that perhaps:
> "Return NULL for relation types that do not appear in partition trees."

Yes.

Thanks!

Stephen

Attachment

pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: pg_partition_tree crashes for a non-defined relation
Next
From: Michael Paquier
Date:
Subject: Re: pg_partition_tree crashes for a non-defined relation