Re: [bug fix] PG10: libpq doesn't connect to alternative hosts when some errors occur

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [bug fix] PG10: libpq doesn't connect to alternative hosts when some errors occur
Date: 2017-05-17 16:34:31
Message-ID: CA+TgmoarkbceFh1E2-_HYgHE+bJ3Axu_tpSRgGxJ1q8ZAPjuqQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, May 17, 2017 at 3:06 AM, Tsunakawa, Takayuki
<tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com> wrote:
> What do you think of the following cases? Don't you want to connect to other servers?
>
> * The DBA shuts down the database. The server takes a long time to do checkpointing. During the shutdown checkpoint, libpq tries to connect to the server and receive an error "the database system is shutting down."
>
> * The former primary failed and now is trying to start as a standby, catching up by applying WAL. During the recovery, libpq tries to connect to the server and receive an error "the database system is performing recovery."
>
> * The database server crashed due to a bug. Unfortunately, the server takes unexpectedly long time to shut down because it takes many seconds to write the stats file (as you remember, Tom-san experienced 57 seconds to write the stats file during regression tests.) During the stats file write, libpq tries to connect to the server and receive an error "the database system is shutting down."
>
> These are equivalent to server failure. I believe we should prioritize rescuing errors during operation over detecting configuration errors.

Yeah, you have a point. I'm willing to admit that we may have defined
the behavior of the feature incorrectly, provided that you're willing
to admit that you're proposing a definition change, not just a bug
fix.

Anybody else want to weigh in with an opinion here?

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2017-05-17 16:38:23 Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run.
Previous Message Tom Lane 2017-05-17 16:31:35 Re: COPY FROM STDIN behaviour on end-of-file