| 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: | 13792.1256664638@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| 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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2009-10-27 17:45:39 | Re: FOR UPDATE versus WITH --- change 8.4 too? |
| Previous Message | Tim Landscheidt | 2009-10-27 17:27:47 | Extraneous newlines in logfile from vacuumdb |