From: | daveg <daveg(at)sonic(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Andrew Chernow <ac(at)esilo(dot)com>, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: libpq Win32 Mutex performance patch |
Date: | 2008-04-11 23:05:06 |
Message-ID: | 20080411230506.GU11030@sonic.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
On Fri, Apr 11, 2008 at 06:25:53PM -0400, Tom Lane wrote:
> Andrew Chernow <ac(at)esilo(dot)com> writes:
> > A more graceful solution would be to print something to stderr and then
> > exit.
>
> stderr doesn't exist, or point to a useful place, in many environments.
> And a forced exit() is no better than a crash for most purposes.
>
> > I don't think libpq should core dump an app by choice.
>
> The case that we are talking about is a bug, or would be a bug if it
> could happen (the fact that we've gotten along happily with no similar
> test in the existing code shows that it can't). Many forms of bug can
> result in core dumps; it's foolish to try to prevent them all. And
> bloating one line of code into five or more lines to defend against
> can't-happen cases is a good way to end up with unreadable,
> unmaintainable software.
>
> regards, tom lane
+1
-dg
--
David Gould daveg(at)sonic(dot)net 510 536 1443 510 282 0869
If simplicity worked, the world would be overrun with insects.
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Chernow | 2008-04-11 23:30:45 | Re: libpq Win32 Mutex performance patch |
Previous Message | Tom Lane | 2008-04-11 22:25:53 | Re: libpq Win32 Mutex performance patch |