pgsql/src/backend/commands (creatinh.c)

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"?

--
Peter Eisentraut peter_e@gmx.net http://yi.org/peter-e/

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