[pgpool-committers: 8341] pgpool: Fix long standing bug with pcp_node_info.

Tatsuo Ishii ishii at sraoss.co.jp
Mon Jan 31 12:35:02 JST 2022


Fix long standing bug with pcp_node_info.

It appears that occasionally pcp_node_info shows backend_status field
as "quarantine" when it should be "down". While pcp_node_info shows
the status, first it checks the backend_status member in BackendInfo
struct. If it is 3, then checks quarantine member. If it is other than
0, then the backend_status field is shown as "quarantine". So if
garbage remains in quarantine member, it is shown as "quarantine". The
BackendInfo struct is transferred from pcp_worker process to pcp
frontend client. Unfortunately when the quarantine member was added by
commit 54af632c, it was forgotten to modify pcp_worker.c and
pcp_frontend.c so that the "quarantine" member is transferred.

Fix is needed to be back patched to 3.7, when the "quarantine" member
was first added.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2022-January/004110.html

Branch
------
master

Details
-------
https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=6d513a1b21047c1d37f53051fb37b4d4b7f1a9f4

Modified Files
--------------
src/libs/pcp/pcp.c       | 6 ++++++
src/pcp_con/pcp_worker.c | 4 ++++
2 files changed, 10 insertions(+)



More information about the pgpool-committers mailing list