Gregory Stark <[EMAIL PROTECTED]> writes:
> I got this assertion failure today:

> postgres=# create table foo (i integer);
> postgres=# alter table foo add j integer;
> postgres=# alter table foo drop j;
> postgres=# create table foo2 () inherits (foo);
> postgres=# explain analyze select * from foo;
> server closed the connection unexpectedly


> I think there isn't actually a bug here except that the assertions are
> incorrect. Everything else in the function looks ok.

In the light of this example, the Assert on new_max_attr is clearly
wrong.  It always was redundant with the ones inside the first loop anyway.
I think the Assert on new_min_attr is still OK though, and in fact
necessary to protect against an array bounds violation in the second
loop.  That Assert is essentially saying that "if the parent table has
OIDs then the child must too", which we do enforce.

Will patch in HEAD and 8.2 --- this code doesn't exist before 8.2.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
       subscribe-nomail command to [EMAIL PROTECTED] so that your
       message can get through to the mailing list cleanly

Reply via email to