From: Bruce Momjian <maillist@candle.pha.pa.us>
To: tgl@sss.pgh.pa.us
Cc: pgsql-interfaces@postgreSQL.org
Cc: pgsql-patches@postgreSQL.org
Subject: Re: [PATCHES] libpq cleanup for SIGPIPE and stderr concerns
Date: Sat, 8 Aug 1998 23:00:27 -0400 (EDT)
Message-ID: <199808090300.XAA29489@candle.pha.pa.us> (raw)
In-Reply-To: <3234.902622955@sss.pgh.pa.us>
> The attached patch implements some changes that were discussed a couple
> weeks ago on the hackers and interfaces lists:
> 1. When the backend sends a NOTICE message and closes the connection
> (typically, because it was told to by the postmaster after another
> backend coredumped), libpq will now print the notice and close the
> connection cleanly. Formerly, the frontend app would usually
> terminate ungracefully due to a SIGPIPE. (I am not sure if 6.3.2
> behaved that way, but the current cvs sources do...)
> 2. libpq's various printouts to stderr are now fed through a single
> "notice processor" routine, which can be overridden by the
> application to direct notices someplace else. This should ease
> porting libpq to Windows.
>
> I also noticed and fixed a problem in PQprint: when sending output
> to a pager subprocess, it would disable SIGPIPE in case the pager
> terminates early (this is good) --- but afterwards it reset SIGPIPE to
> SIG_DFL, rather than restoring the application's prior setting (bad).
>
> regards, tom lane
>
Patch applied.
--
Bruce Momjian | 830 Blythe Avenue
maillist@candle.pha.pa.us | Drexel Hill, Pennsylvania 19026
+ If your life is a hard drive, | (610) 353-9879(w)
+ Christ can be your backup. | (610) 853-3000(h)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-interfaces@postgresql.org
Cc: maillist@candle.pha.pa.us, tgl@sss.pgh.pa.us, pgsql-patches@postgreSQL.org
Subject: Re: [PATCHES] libpq cleanup for SIGPIPE and stderr concerns
In-Reply-To: <199808090300.XAA29489@candle.pha.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox