Re: libpq and psql not on same page about SIGPIPE - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: libpq and psql not on same page about SIGPIPE
Date
Msg-id 200412012242.iB1Mgre00247@candle.pha.pa.us
Whole thread Raw
In response to libpq and psql not on same page about SIGPIPE  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Manfred Spraul wrote:
> >This seems workable as long as we document the possible gotchas.
> >
> >  
> >
> Is that really worthwhile? There are half a dozend assumption about the 
> C library and kernel internal efficiency of the signal handling 
> functions in the proposal. Adding a PQinitLib function is obviously a 

The main path uses pthread_sigmask() and sigpending().  Are those
possible performance problems?  I see how signal() would be a thread
problem, but not those.

> larger change, but it solves the problem.
> I'm aware of one minor gotcha: PQinSend() is not usable right now: it 
> relies on the initialization of pq_thread_in_send, which is only created 
> in the middle of the first connectDB(). That makes proper signal 
> handling for the first connection impossible.

I think that whole PQinSend thing is pretty ugly, even if I did write
it.  My current patch seems like a great improvement.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: New compile warnings for inheritance
Next
From: Tom Lane
Date:
Subject: Re: nodeAgg perf tweak