Skip site navigation (1) Skip section navigation (2)

Re: Delete cascade with three levels bug ?

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Euler Taveira de Oliveira <euler(at)timbira(dot)com>, Marcelo Costa <marcelojscosta(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Delete cascade with three levels bug ?
Date: 2009-10-27 17:30:38
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Tue, Oct 27, 2009 at 10:41 AM, Euler Taveira de Oliveira
>> BTW, is it worth preventing such a crash putting an elog message in trigger.c?

> It doesn't seem right to allow a catalog change that results in an
> assertion failure.  Seems like we should either prevent the catalog
> change, or have an elog() there rather than Assert().

We can't prevent a superuser from making stupid catalog changes.  I
agree though that it's not very nice to Assert or dump core afterwards.
Changing the assert to an elog seems like an appropriate answer,
assuming that this isn't masking some more significant issue.

			regards, tom lane

In response to

pgsql-hackers by date

Next:From: Robert HaasDate: 2009-10-27 17:45:39
Subject: Re: FOR UPDATE versus WITH --- change 8.4 too?
Previous:From: Tim LandscheidtDate: 2009-10-27 17:27:47
Subject: Extraneous newlines in logfile from vacuumdb

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group