From: Alvaro Herrera from 2ndQuadrant <alvherre@alvh.no-ip.org>
To: Haribabu Kommi <kommi.haribabu@gmail.com>
Cc: Tsunakawa, Takayuki <tsunakawa.takay@jp.fujitsu.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Dave Cramer <pg@fastcrypt.com>
Cc: Jing Wang <jingwangian@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: Libpq support to connect to standby server as priority
Date: Tue, 10 Sep 2019 13:05:29 -0300
Message-ID: <20190910160529.GA11336@alvherre.pgsql> (raw)
In-Reply-To: <20190909171949.GA31491@alvherre.pgsql>
On 2019-Sep-09, Alvaro Herrera from 2ndQuadrant wrote:
> Question about 0001. In the CONNECTION_SETENV state, you end by falling
> through to the CONNECTION_CHECK_TARGET case; but in that switch it seems
> a bit unnatural to do that. I think doing "goto keep_trying" is another
> way of doing the same thing, but more in line with what every other
> piece of code does.
Appears to work. Pushed like that.
Testing protocol version 2 is difficult! Almost every single test fails
because of error messages being reported differently; and streaming
replication (incl. pg_basebackup) doesn't work at all because it's not
possible to establish replication connections. Manual inspection shows
it behaves correctly.
Remaining patchset attached (per my count it's v13 of your patchset.
Please use "git format-patch -v14" and so on when posting patches). I
stripped the doc change from 0001 (unchanged) because I found it hard to
justify on its own, and it has a couple of grammatical mistakes. I
think we should merge one half of it with each of the other two patches
where the changes are introduced (0003 and 0004). I'm not convinced
that we need 0004-0006 to be separate commits.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
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-hackers@postgresql.org
Cc: alvherre@alvh.no-ip.org, kommi.haribabu@gmail.com, tsunakawa.takay@jp.fujitsu.com, robertmhaas@gmail.com, laurenz.albe@cybertec.at, tgl@sss.pgh.pa.us, pg@fastcrypt.com, jingwangian@gmail.com
Subject: Re: Libpq support to connect to standby server as priority
In-Reply-To: <20190910160529.GA11336@alvherre.pgsql>
* 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