Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Patch applied. Thanks.
> Did you pay no attention to the subsequent discussion?
I thought that was related to a different patch but now I see it was the
> While I don't object to renaming the routines internally, I do have
> strong doubts about changing the externally-visible error messages.
> I'd suggest undoing the particular changes that pass routine names
> to ExecConstraints, so that the error messages stay the same. We
> can clean it up at some time *after* we offer error codes that clients
> can test.
Well, with no error codes on the horizon, and schemas appearing to break
lots of stuff, I don't see the need to keep error messages consistent.
Heck, the error messages says:
elog(WARNING, "ExecReplace: replace can't run without transaction");
and we haven't had replace since 1994 or so. I think the cleanup is
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
In response to
pgsql-patches by date
|Next:||From: Bruce Momjian||Date: 2002-06-25 18:09:54|
|Subject: Re: Page access|
|Previous:||From: Bruce Momjian||Date: 2002-06-25 17:58:10|
|Subject: Re: several minor cleanups|