Re: Libpq support to connect to standby server as priority
Tatsuo Ishii <ishii@sraoss.co.jp>
From: Tatsuo Ishii <ishii@sraoss.co.jp>
To: pg@fastcrypt.com
Cc: ishii@sraoss.co.jp, tsunakawa.takay@jp.fujitsu.com, tgl@sss.pgh.pa.us,
robertmhaas@gmail.com, laurenz.albe@cybertec.at, kommi.haribabu@gmail.com,
jingwangian@gmail.com, pgsql-hackers@postgresql.org
Date: 2019-01-18T00:15:24Z
Lists: pgsql-hackers
> On Thu, 17 Jan 2019 at 18:03, Tatsuo Ishii <ishii@sraoss.co.jp> wrote: > >> > On Wed, 16 Jan 2019 at 01:02, Tatsuo Ishii <ishii@sraoss.co.jp> wrote: >> > >> >> > From: Tatsuo Ishii [mailto:ishii@sraoss.co.jp] >> >> >> But pg_is_in_recovery() returns true even for a promoting standby. So >> >> >> you have to wait and retry to send pg_is_in_recovery() until it >> >> >> finishes the promotion to find out it is now a primary. I am not sure >> >> >> if backend out to be responsible for this process. If not, libpq >> would >> >> >> need to handle it but I doubt it would be possible. >> >> > >> >> > Yes, the application needs to retry connection attempts until success. >> >> That's not different from PgJDBC and other DBMSs. >> >> >> >> I don't know what PgJDBC is doing, however I think libpq needs to do >> >> more than just retrying. >> >> >> >> 1) Try to find a node on which pg_is_in_recovery() returns false. If >> >> found, then we assume that is the primary. We also assume that >> >> other nodes are standbys. done. >> >> >> >> 2) If there's no node on which pg_is_in_recovery() returns false, then >> >> we need to retry until we find it. To not retry forever, there >> >> should be a timeout counter parameter. >> >> >> >> >> > IIRC this is essentially what pgJDBC does. >> >> Thanks for clarifying that. Pgpool-II also does that too. Seems like a >> common technique to find out a primary node. >> >> > Checking the code I see we actually use show transaction_read_only. > > Sorry for the confusion So if all PostgreSQL servers returns transaction_read_only = on, how does pgJDBC find the primary node? Best regards, -- Tatsuo Ishii SRA OSS, Inc. Japan English: http://www.sraoss.co.jp/index_en.php Japanese:http://www.sraoss.co.jp
Commits
-
Make test_target_session_attrs more robust against connection failure.
- 3769e11a3183 14.0 landed
-
Extend the abilities of libpq's target_session_attrs parameter.
- ee28cacf619f 14.0 landed
-
Mark default_transaction_read_only as GUC_REPORT.
- d16f8c8e416d 14.0 landed
-
Introduce a new GUC_REPORT setting "in_hot_standby".
- bf8a662c9afa 14.0 landed
-
Avoid spamming the client with multiple ParameterStatus messages.
- 2432b1a04087 14.0 landed
-
Avoid harmless Valgrind no-buffer-pin errors.
- a766d6ca22ac 14.0 cited
-
Restructure libpq code to remove some duplicity
- b438e7e7a1c5 13.0 landed
-
libpq: Add target_session_attrs parameter.
- 721f7bd3cbcc 10.0 cited