Re: [PATCH] Log crashed backend's query (activity string) - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [PATCH] Log crashed backend's query (activity string)
Date
Msg-id CA+Tgmobjp4zgy-vRznJ2NEZJBjgfTXEP1PRZnAQrgNfW=mJkaQ@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Log crashed backend's query (activity string)  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCH] Log crashed backend's query (activity string)
List pgsql-hackers
On Wed, Sep 7, 2011 at 4:15 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> On Wed, Sep 7, 2011 at 3:42 PM, Alvaro Herrera
>> <alvherre@commandprompt.com> wrote:
>>> A mishandled encoding conversion could be problematic, so that needs to
>>> be carefully considered (perhaps just shut off unconditionally).
>
>> It's not really a problem for the postmaster if something just plain
>> old fails.  Where we get into trouble is if it manages to (a) crash,
>> (b) take an excessive amount of time to complete, or (c) screw up the
>> postmaster state in some way we can't recover from.  But if any of
>> those are an issue then, yeah, just shut it off.
>
> Keep in mind that in the postmaster, elog(ERROR) *is* a crash.

Right... but a function that returns -1 to indicate that something
didn't work should be OK, as long as whatever it does is otherwise
extremely boring.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: FATAL: lock AccessShareLock on object 0/1260/0 is already held
Next
From: Marti Raudsepp
Date:
Subject: Re: [PATCH] Don't truncate integer part in to_char for 'FM99.'