On Mon, Feb 1, 2010 at 1:31 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> I have cleaned up and simplified this patch. Attached is the version
>> I intend to commit. Changes:
> Minor suggestions:
> I think the names like "rel_parents" would read better as
> "rel_numparents" etc. As-is, the reader could be forgiven for expecting
> that this will be a list of parent relation OIDs or some such.
I thought about that but ended up being lazy and leaving it the way I
had it. I'll go be un-lazy.
> The new loop added within find_all_inheritors could really do with an
> addition to the comments, along the line of "If a child is already
> seen, increment the corresponding numparents count".
> I don't trust the proposed "order by attrelid" business in the
> regression test --- once in a blue moon, that will fail because the
> OID counter wrapped around mid-test, and we'll get an unreproducible
> bug report. I'd suggest order by attrelid::regclass::text.
Wow, didn't think of that. Will change.
> Looks sane otherwise.
Thanks for the review.
In response to
pgsql-hackers by date
|Next:||From: Robert Haas||Date: 2010-02-01 18:48:25|
|Subject: Re: remove contrib/xml2|
|Previous:||From: Tom Lane||Date: 2010-02-01 18:38:07|
|Subject: Re: remove contrib/xml2 |