On Thu, Sep 9, 2010 at 20:23, Tom Lane <t...@sss.pgh.pa.us> wrote:
> Magnus Hagander <mag...@hagander.net> writes:
>> On Thu, Sep 9, 2010 at 19:48, Tom Lane <t...@sss.pgh.pa.us> wrote:
>>> On balance I think I'd suggest an #ifdef WIN32 in CleanupBackend that
>>> made it accept 128 as a "normal exit" case.
>
>> Seems reasonable. I'll whack it around for that - see attached.
>
> Hm, still doesn't log, which I think it should, even for testing
> purposes (how will you know the case occurred?).  Maybe like this:

Agreed, that's better.

>> Dave has a reasonably reproducible test environment. Unforunately it's
>> on 8.3, so this patch will be completely unsafe there (it doesn't have
>> the deadman switch). But hopefully it can be used to see it fixes this
>> problem (while introducing others)h
>
> Sounds like a plan.

Patch is with dave for testing now :-)


-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to