Re: pgsql/src/backend/commands (creatinh.c) - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql/src/backend/commands (creatinh.c)
Date
Msg-id 5127.976826683@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql/src/backend/commands (creatinh.c)  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-committers
Peter Eisentraut <peter_e@gmx.net> writes:
> tgl@postgresql.org writes:
>> Change StoreCatalogInheritance() to work from a list of parent relation
>> OIDs rather than names.  Aside from being simpler and faster, this way
>> doesn't blow up in the face of 'create temp table foo () inherits (foo)'.
>> Which is a rather odd thing to do, but it seems some people want to.

> Just wondering, is the "odd" thing here the 'temp', the empty column list,
> or 2 times "foo"?

The latter.  The failure case was

    create table foo (...);
    create temp table foo (...) inherits (foo);

the problem being that the temp table name mapping for the temp foo
was installed before we were done looking up its ancestors.

            regards, tom lane

pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: pgsql/src/backend/commands (creatinh.c)
Next
From: tgl@postgresql.org
Date:
Subject: pgsql/src/backend/optimizer/path (allpaths.c indxpath.c joinpath.c pathkeys.c)