Re: [patch]socket_timeout in interfaces/libpq - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: [patch]socket_timeout in interfaces/libpq
Date
Msg-id 20190910063821.GE11737@paquier.xyz
Whole thread Raw
In response to RE: [patch]socket_timeout in interfaces/libpq  ("nagaura.ryohei@fujitsu.com" <nagaura.ryohei@fujitsu.com>)
Responses Re: [patch]socket_timeout in interfaces/libpq
RE: [patch]socket_timeout in interfaces/libpq
List pgsql-hackers
On Wed, Jun 26, 2019 at 11:56:28AM +0000, nagaura.ryohei@fujitsu.com wrote:
> It seems that you did not think so at that time.
> # Please refer to [1]
>
> I don't think all the reviewers are completely negative.

I recall having a negative impression on the patch when first looking
at it, and still have the same impression when looking at the last
version.  Just with a quick look, assuming that you can bypass all
cleanup operations normally taken by pqDropConnection() through a
hijacking of pqWait() is not fine as this routine explicitely assumes
to *never* have a timeout for its wait.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Replication & recovery_min_apply_delay
Next
From: "Tsunakawa, Takayuki"
Date:
Subject: RE: [bug fix] Produce a crash dump before main() on Windows