Fix memory leak pointed out by Coverity.

Actually it's a false positive.

Fix health check process issues pointed out by Coverity.

Fix possible NULL terminate missing and memory leak when running in test mode.

Adjusting the field name in pcp_watchdog_info.

Details in: https://www.pgpool.net/pipermail/pgpool-hackers/2021-December/004070.html

Enhance parameter status handling.

When a parameter status message arrives from backend, Pgpool-II
memorized it but did not forward to frontend.  This commit allows
forwarding a parameter status message to frontend.

Revert changes accidentally included in commit f9521fe4.

Commit f9521fe4 (Implement flush tracking feature) accidentally
included changes for Parameter status change fix patch proposed in
[pgpool-hackers: 4103] Re: What to do with ParamterStatus?
https://www.pgpool.net/pipermail/pgpool-hackers/2022-January/004104.html

A commit should only include single change. Unrelated changes should
not brought in together. So revert the part (Parameter status change
fix).

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

Fix comment in libpcp_ext.h.

Fix comment for standby_delay_by_time. In the comment unit used in
standby_delay is microseconds, not milliseconds when
standby_delay_by_time is true.

Change the way to obtain replication delay when delay_threshold_by_time is specified.

Use pg_stat_replication.replay_lag.  This way makes the code much
simpler and more precise replication delay can be obtained. The only
downside is pg_stat_replication.replay_lag is only available in
PostgreSQL 10 or later (previous method can be used in 9.5 or later).
I think the down side is not worth the trouble and we should use
pg_stat_replication.replay_lag.

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

Allow to specify replication delay by time.

delay_threshold specifies replication delay upper limit in bytes. Add
similar parameter called delay_threshold_by_time so that the limit can
be specified by time (seconds). The new parameter is effective if it is
greater than 0 and track_commit_timestamp (available in >> PostgreSQL
9.5 or after) is enabled. In this case "show pool_ndoes" and
pcp_node_info display the replication delay in seconds. If the
parameter is set to 0 or track_commit_timestamp is not enabled,
delay_threshold_by_time is ignored and falls back to delay_threshold
mode.

For this purpose new member standby_delay_by_time is added to shared
memory data Backendinfo to distinguish whether replication delay is
measured in byte (standby_delay_by_time == false) or seconds
(standby_delay_by_time = true). If standby_delay_by_time is true,
standby_delay is measured in second * 1000000, so that the precision
is 6 digits after decimal point.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-December/004081.html

Fix pcp_node_info hang when pgpool cannot connect to backend.

Since 4.3 pcp_node_info (and show pool_nodes) try to connect to all
backend to obtain real-time and actual backend status.  In certain
cases connect(2) fails and keeps on retrying, and the command does not
complete. To fix this, following modifications are made:

1) db_node_status(), which is responsible for probing backend is alive
or not, tries to connect backend with "connect_timeout" parameter
enabled. The timeout is taken from "connect_timeout" parameter of
pgpool.conf. If connect_timeout in pgpool.conf disabled (i.e. set to
0), the timeout parameter is not set. In this case the command will
not complete.

2) db_node_role(), which is responsible for fetching backend role (primary
or standby), does to not retry in connecting to backend using
make_persistent_db_connection_noerror().

3) inform_node_info(), which is the workhorse of pcp_node_info, tries
to fetch info (thus calling db_node_status() and db_node_role()) only
for specified backend. Before it unconditionally access all backend.

Problem reported and patch reviewed by Emond Papegaaij.

Discussion: https://www.pgpool.net/pipermail/pgpool-general/2022-January/008042.html

Add an extended query protocol test for flush tracking.

Implement flush tracking feature.

When a flush message arrives from frontend, any pending message from
backend should be flushed and sent to frontend. In order to do that,
this commit implements "flush tracking" feature. i.e. when a flush
message arrives, pgpool sets "flush pending" flag in each pending
messages. If the response message from backend corresponds to the
pending message with the flush pending flag being set, the message is
immediately flushed to frontend, rather than buffered.

Discussion: https://www.pgpool.net/pipermail/pgpool-general/2022-January/008026.html

Enhance pgproto.

Allow to show ParameterStatus's parameter name and value.

Change the default value for log_line_prefix.

Currently the default value for log_line_prefix is "%t pid: %p", which
is the long standing default. Also since 4.2, %a (application name) and
since 4.3 %m (timestamp with millisecond) are available.

This commit changes %t" (timestamp without millisecond) to %m because
PostgreSQL's default is already %m.  Also add %a so that admins easily
distinguish particular process's log.

In summary the default is changed from:
"%t pid: %p"
to:
"%m %a pid: %p"

This change will be applied to not only master branch but 4.3
stable. There's not so many 4.3 users yet and changing the default
will not affect existing 4.3 users. Only new 4.3 will enjoy the new
default.

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

Fix for a small mistake in pgpool-recovery SQL script

Fix bug of wd_no_show_node_removal_timeout.

Rename "wd_initial_node_showup_time" to "wd_no_show_node_removal_timeout".

Test: fix pgpool_setup and watchdog_setup binary PATH in noinstall mode.

Update Makefile.in

Fix regression test 075.

The test reported success even if pgpool does not start up.
Problem reported and patch provided by Qiang Lingjie.
Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2022-January/004086.html

Fix bug of pgpool_remote_start.sample.

Doc: enhance pgproto.

Add documents for the previous commit.

Enhance pgproto.

Add new command 'z'. This is similar to 'y' except 'z' reads only 1
message and do not wait for "ready for query" arrives (or timeout if
no message arrives within 1 second).

Fix compiler warning.

In the commit "Suppress message length log for in_hot_standby." I
forgot to push a modification to
src/include/protocol/pool_proto_modules.h which caused a compiler
warning.

Doc: add restriction regarding ParameterStatus and in_hot_standby parameter.

Suppress message length log for in_hot_standby.

PostgreSQL 14 introduced new config parameter: in_hot_standby
https://www.postgresql.org/docs/14/runtime-config-preset.html
The value is either "on" for standby servers or "off" for primary
servers. As a result pgpool log is fladded by the messages:

2021-12-16 10:40:34.855: psql pid 366965: LOG:  reading message length
2021-12-16 10:40:34.855: psql pid 366965: DETAIL:  message length (22) in slot 1 does not match with slot 0(23)

To avoid this, only complain if the parameter name is not in_hot_standby.
Also the message is enhanced to show the parameter name.

2022-01-05 13:05:15.993: psql pid 642877: LOG:  ParameterStatus "TimeZone": node 1 message length 30 is different from main node message length 24

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-December/004077.html

Doc: add "exclude" settings to /etc/yum.repos.d/pgdg-redhat-all.repo so that Pgpool-II is not installed from PostgreSQL YUM repository.

Doc: fix documentation typos.

Patch is created by Umar Hayat.

Doc: fix release notes.

Add release notes.

Allow to run regression test against existing installation without recompiling.

It is possible to run regression test using existing installation.

regression.sh -m noinstall

However some of tests fail in this case because those tests require
pgpool to be compiled with variable HEALTHCHECK_DEBUG is set. This is
only possible in following procedure.

make clean
cd src/test/regression
./regress.sh

To run the regression test against existing installation new config
variable "health_check_test" is added. The source code is always
compiled as if HEALTHCHECK_DEBUG is set. The test facility is not
activated unless health_check_test is set to on.

For now I push to only master branch. After some tests, I am going to
push to all supported branches. I know adding new parameter to stable
branches is unusual but the feature is for enhancing test and it is
not visible to ordinal users. So I think my plan is justified by them.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-December/004078.html

Revert "Add validations of wd_lifecheck_password and recovery_password format"

This reverts commit 0f587d1741f20140ea8a2293b40946de27aec736.

This commit caused failure in regression test due to pcp_recovery_node_error:
recovery node 1...ERROR:  invalid password format for recovery_user: t-ishii
DETAIL:  md5 hashed password is not allowed here

Add validations of wd_lifecheck_password and recovery_password format

wd_lifecheck_password and recovery_password are not allowed to be md5
hashed password format but pgpool did not check them.

Doc: sync with English manual.

Doc: fix typo in pcp_watchdog_info manual.

Doc: fix typos

Patch is created by Lu Chenyang.

Doc: update japanese configuration example for SR mode.

Doc: update doc version.

Update Makefile.

Update Makefile.

Doc: update docs.

Doc: add mention about add native/SI mode example to 4.3 release note.

Rename recovery_2nd_stage.sample to replication_mode_recovery_2nd_stage.sample

Update pgpool.spec.

Doc: add new configuration example for replication mode and si mode.

Doc: update documentation "Pgpool-II + Watchdog Setup Example".

Suppress bison warnings regarding yacc incompatibility.

Run bison without yacc compatibility may raise some risks, so just suppress warnings.

Doc: fix typos.

Add 4.3RC1 release notes.

Fix mention about insert_lock in 4.3 release note.

The compile default was changed twice while preparing 4.3.  First, on
to off, then finally off to on. To avoid the confusion, remove "on to
off" part.

Fix redundant code.

Patch contributed by Lu Chenyang.

Add release notes for Pgpool-II 4.2.6, 4.3beta2.

Reject extraneous data after SSL encryption handshake.

In the server side implementation of SSL negotiation
(pool_ssl_negotiate_serverclient()), it was possible for a
man-in-the-middle attacker to inject arbitrary SQL commands. This is
possible if Pgpool-II is configured to use cert authentication or
hostssl + trust. This resembles PostgreSQL's CVE-2021-23214.

Similarly, in the client side implementation of SSL negotiation
(pool_ssl_negotiate_clientserver()), it was possible for a
man-in-the-middle attacker to inject arbitrary responses. This is
possible if PostgreSQL is using trust authentication with a clientcert
requirement. It is not possible with cert authentication because
Pgpool-II does not implement the cert authentication between Pgpool-II

To fix these reject extraneous data in the read buffer after SSL
encryption handshake.
and PostgreSQL. This resembles PostgreSQL's CVE-2021-23222.

Deal with PostgreSQL 14 while processing pg_terminate_backend().

Do not reject two arguments form of pg_terminate_backend() as
PostgreSQL 14 or after accept two arguments.

Fix occasional 073.pg_terminate_backend regression test failure.

The test used "ps -ef" command to find the process which is running
SELECT command.  However in some cases the "ps -ef" command omits part
of "SELECT" in its output and this made the test fail.
So use "ps -efw" instead of "ps -ef" to prevent it.

Doc: update 4.3 release note.

Suggestion from Muhammad Usama.

Doc: add Japanese translation for commit a683288045332fdd8ea7cd7038510353cb3dbfc

Rename regression test 074.

Update patch file.

Doc: Mention of risks with using the dynamic cluster membership.

Doc: update docs.

Update version to Pgpool-II 4.4 devel.

Enable AM_MAINTAINER_MODE on master branch.

Disable AM_MAINTAINER_MODE.

Doc: fix typo.

Doc: translate into Japanese some of 4.3 release note.

Doc: add mention about failover_on_backend_shutdown to the migration section in Japanese doc.

Also translate into Japanese new documents added in:
https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=4760459d45e33b573458e3973e192c43077f7eb1

Doc: add new watchdog dynamic cluster definition.

Doc: remove unnecessary character from wd_remove_shutdown_nodes.

Doc: add 4.3 Japanese release note.

doc: Documentation update for dynamic cluster membership feature

[New-Feature] Enable dynamic cluster membership of watchdog nodes

Enables dynamic watchdog cluster definition by introducing the concept of
Member and Nonmember watchdog nodes. If the node's membership gets revoked
from the watchdog cluster, then the cluster re-calibrate itself dynamically
to adjust all subsequent majority rule computations.

Three new settings wd_no_show_node_removal_timeout, wd_lost_node_removal_timeout
and wd_remove_shutdown_nodes are added to configure the membership
criteria of the watchdog nodes.

Doc: add mention about failover_on_backend_shutdown to the migration section.

Doc: fix description on pgpool.conf.sample-*

This is a follow up on commit:
https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commit;h=4ab1566ff133e449a4ddff7f1d7133b87d10152d

Doc: mention that the default of failover_require_consensus has been changed to on.

Change compile time default value of failover_require_consensus to on.

failover_require_consensus is already on in the docs and in
pgpool.conf.sample-stream. So sync with them.

Remove sample config files from Makefile, because the files has been removed by the previous commit.

Doc: mention that snapshot isolation mode is now ready for production in the 4.3 release note.

Doc: remove descriptions on pgpool.conf.sample-*

This is a follow up on commit:
https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commit;h=4ab1566ff133e449a4ddff7f1d7133b87d10152d

Doc: initial cut of Pgpool-II 4.3 release note.

Still some ongoing items are missing.

Unify all pgpool.conf.sample files into single pgpool.conf.sample.

Now that all configuration parameters in pgpool.conf are commented out,
there's no point to have separate configuration sample files. Just
specify the backend_clustering_mode parameter is enough. pgpool_setup
is modified to adopt the change.

Update to documents will be followed.

Fix application_name array lacking an entry for logger process.

Add logger process entry to the process name array. This was missed
since the logger process was added.
Per coverity.

Enhance SIGHLD handler of Pgpool-II main process.

When Pgpool-II child is killed by SIGKILL signal, the SIGHLD handler
just emitted LOG level message as other signals.  But SIGKILL is an
important event, for example killed by OOM killer. So emit a WARNING
level message instead.

Per suggestion from Michail Alexakis.
Discussion: https://www.pgpool.net/pipermail/pgpool-general/2021-October/007808.html

Fix connection counter issue when reserved_connections is 0.

If reserved_connections is 0, we don't need to manage the connection
counter to check if the count is larger than
(pool_config->num_init_children - pool_config->reserved_connections).
So remove the check.  This will prevent unwanted "Sorry, too many
clients already" error" by accidental counter leak.

For reserved_connections > 0 case, we need to fix the counter leak but
it's another story.

Discussion: https://www.pgpool.net/pipermail/pgpool-general/2021-October/007808.html

Fix missing comment out in sample pgpool.conf.

statement_level_load_balance and allow_clear_text_frontend_auth were
not commented out.

Doc: update Japanese "Pgpool-II + Watchdog Setup Example" and sample scripts.

Updating Japanese document and script to Pgpool-II 4.3 and
Postgresql 14. English document will updated later.
Pgpool-II 4.3 yum repository url is blank and sample script
link is set master branch because there are still not existed.
We will update those when Pgpool-II 4.3 release.

Commenting out all parameters in pgpool.conf.

This will make pgpool.conf less confusing and make PostgreSQL users
easier to understand pgpool.conf because this follows the PostgreSQL
way.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-October/004044.html

Change compile time default value of insert_lock to on.

insert_lock is already on in the in pgpool.conf.sample-replication and
pgpool.conf.sample-snapshot. So sync with them. Documentations are
changed as well.

Change compile time default value of failover_when_quorum_exists to on.

failover_when_quorum_exists is already on in the docs and in
pgpool.conf.sample-stream. So sync with them.

Change compile time default value of load_balance_mode to on.

load_balance_mode is already on in pgpool.conf.sample. So sync with it.

Doc: fix documentation typos.

Enable streaming replication check by default.

Change the compile time default value of sr_check_period from 0 to
10. This means if operating in streaming replication mode, now
streaming replication check is enabled by default. The parameter is
already set to 10 in pgpool.conf.sample-stream. So the only difference
is, when sr_check_period is commented out.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-October/004042.html

Fix there's no new line at the end of file.

Fix nasty bug with pgpool_recovery() in 4.3 or later.

Reported by Kawamoto and Peng Bo.

Fix signal handler for SIGTERM, SIGINT and SIGQUIT.

It did not properly save errno and it was possible to overwrite errno.

Change compile time default of log_standby_delay from 'none' to 'if_over_threshold'.

Before the compile time default was 'none' but the default value of
sample pgpool.conf was 'if_over_threshold'. Since 'if_over_threshold'
is more useful for most users, change the compile time default to
'if_over_threshold' as well.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-October/004035.html

Fix pcp_proc_info, show pool_processes and show pool_pools command.

I added new fields to the result of those commands in following commit:
https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commitdiff;h=45152975fdf8d57deaff07dc6c34117fa8eeb3db

The usage of client_idle_limit was mistaken. Child processes are
restarted by child_life_time not client_idle_limit, and client_idle_limit
is related to Client idle duration and disconnects the connection.

So, I changed the display of Start time and Client idle duration.
I modify wait_for_connections function to calculate the time remaining
until the child process is restarted by child_life_time; Instead of
using child_life_time as the timeout for select(2), select(2) with 1 sec
timeout is looped through child_life_time times to get the elapsed time.

Moreover when there were multiple backends, Client idle duration was
only displayed in backend0, so I fix this too.

Doc: fix documentation typos.

Fix typos in documentation and sample scripts.

Patch is created by Kazufumi Noto.

Add missing files to tar ball.

Doc: fix yum install command typo in configuration example.