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.

Fix some compile time default values.

Some of default values defined in src/config/pool_config_variable.c
were different from what the documentation expects. Namely:

insert_lock (changed from "true" to "false")
clear_memqcache_on_escalation (changed from "false" to "true")
wd_lifecheck_dbname (changed from "postgres" to "template1")
listen_backlog_multiplier (changed from "32" to "2")
authentication_timeout (changed from "0" to "60")

Fix for bug-732: Segmentation fault at failover ...

trigger_failover_command() had an assumption that old primary can never be NULL,
which of course is not the case.

Feature: Import PostgreSQL 14 beta2 new parser.

Major changes of PostgreSQL 14 parser include:

- Allow an alias to be used to a JOIN ... USING

  SELECT ... FROM t1 JOIN t2 USING (a, b, c) AS x

- Allow DISTINCT to be added to GROUP BY

  SELECT c1, max(c2) FROM t1 GROUP BY DISTINCT c1;

- New SEARCH and CYCLE clauses for common table expressions

Fix pg_config command path to avoid test failure.

Fix to support PostgreSQL version (e.g. alpha, beta).

Fix psql command path to avoid test failure.

Add new config parameter "failover_on_backend_shutdown".

If set to on, admin shutdown triggers failover (existing behavior).
If set to off, admin shutdown does not trigger failover (new
behavior), and is useful to completely avoid unwanted failover caused
by pg_terminate_backend() and SIGTERM signal sent to the backend
process.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-August/003999.html

Fix occasional hang in COPY FROM.

If an error occurs while doing COPY FROM, it was possible the
Pgpool-II waited forever for a response from backend after COPY end
marker was sent from frontend. Pgpool expected a new message arrives
to socket, but it is possible that the message (in this case an error
message) is already in the backend read buffer. The fix is, check the
buffer is empty or not before reading from the socket.
New test case (07.copy_hang) is also added.

The bug was found by Bo Peng.

Doc: fix incorrect file name in "Pgpool-II on Kubernetes".

Doc: add release notes for Pgpool-II 4.2.5.

Fix for bug-731: Fails to execute follow_primary_command..

It was a segmentation fault happening because of a silly coding mistake in
parse_wd_exec_cluster_command_json() function.
The problem was the WDExecCommandArg** arg for returning the argument list
was not getting populated properly.
As part of the fix, to get rid of complexity and confusion, I have changed
the function signature to accept List* type argument for returning the
parsed command args instead of an array and its length.

Thanks to Bo Peng, Emond Papegaaij and Tatsuo Ishi for reviewing
and testing the fix

Add new fields to pcp_proc_info, show pool_processes, show pool_pools command.

The purpose of this commit is to get more information about the process stats
and how pgpool is working with "client_idle_limit" and "child_max_connections".

Start time:
I add “remaining” that is the remaining time beforl the process is restarted by
client_idle_limit.

Client connection count:
The connection count from clients to this pgpool process. Useful to know how
many times clients can connect to this pgpool process before
child_max_connections reaches.

Backend connection time:
I just rename create_time for clarity.

Client connection time:
The time when the client last connected. If the client uses its own connection
pooling, this helps to know when the connection was actually made from pgpool's
point of view.

Client idle duration:
The elapsed time (in seconds) since the client became in idle state.

Client disconnection time:
This is the last time the client disconnects. The use case is the same as above.

Status:
This is the process status. This shows 4 status: “Wait for connections”, “Idle”,
“Idle in transaction”, “Execute command”.

Doc: fixed "PGPOOL SHOW" documentaion that was missing in the previous commit 151e8f54961b1a0394cfc86c2f4ddf715b41ceea.

Fix incorrect PGPOOL SHOW option.

Since 4.2 other_pgpool is removed, "PGPOOL SHOW other_pgpool" command should be changed to "PGPOOL SHOW watchdog".

Fix bug of "PGPOOL SHOW heartbeat" and "PGPOOL SHOW ALL" command.

The last member in heartbeat_* array was not displayed in the command result.

Fix to allow log_rotation_age to be set to 0.

According to the docs, it is possible to set 0 to log_rotation_age so
that the log rotation feature is disabled. But the allowed minimum
value for it was mistakenly set to 10, which made it impossible to set
log_rotation_age to 0.

Support include directive in pgppol.conf file

The pgpool.conf can be nested by 'include' directive.A depth of include have a
restriction for avoiding loop back of include files. 'include' is not regular
parameter, so SHOW command don't display it.

Fix pgpool_setup to do nothing when no new main node is available.

When no new main node is available due to there's only 1 node
remaining, pgpool_setup tried to promote a node with node id -1.  This
is impossible. Fix is just skipping whole failover process if new main
node id is -1.

Fix the incorrect display of load balancing node in raw mode.

In raw mode, Pgpool-II sends all queies to main node.
This is harmless, but it may confuse users.

Fix pgpool logger process eats 100% cpu.

The select(2) loop in the logger process did not initialize
timeout.tv_usec, which could cause error in select(2) because of too
big tv_usec. In this select(2) immediately returned and iterate the
for loop.

Problem reported and patch provided by Fang Jun.
Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-August/003993.html

Fix compiler warning.

Update Makefile.in and configure.

Fix "ar: `u' modifier ignored since `D' is the default (see `U')" warnings.

The warning can be eliminated by changing arguments of ar command from
"cru" to "cr".  There were two places of the warnings: src/libs/pcp
and src/watchdog.  For the former, we need to fix libtool, which is
generated by configure. So fix configure.ac.  For the latter, just fix
src/watchdog/Makefile.am.

Update configure and src/config/pool_config.c due to the previous commit.

Fix compiler warnings generated by newer version of gcc.

gcc on Ubuntu 20 (9.3.0) generates tons of compiler warnings something like:

In file included from /usr/include/string.h:495,
>                  from pg_md5.c:31:
> In function 。strncpy「,
>     inlined from 。update_pool_passwd_from_file「 at pg_md5.c:276:3,
>     inlined from 。main「 at pg_md5.c:136:3:
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:10: warning: 。__builtin___strncpy_chk「 output may be truncated copying between 0 and 128 bytes from a string of length 257 [-Wstringop-truncation]
>   106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
>       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>

I have found a message in PostgreSQL hackers mailing list regarding similar issue:
https://www.postgresql.org/message-id/flat/21789.1529170195%40sss.pgh.pa.us#525c384d13505fa6f1f25c50b00d7a08

To fix these warnings:

1) fix misuse of strncpy() and friends

2) add -Wno-stringop-truncation to CLFAG

This commit implelents #1 and #2.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-August/003990.html

Fix SI mode to acquire a snapshot with an internal transaction.

In SI mode a data modifying statement needs to start internal
transaction.  However it forgot to acquire a snapshot. This commit
fixes that in both simple query protocol and extended query protocol.

Update Makefile and SPEC file to include "pgpool_recovery--1.3--1.4.sql" and "pgpool_recovery--1.4.sql".

Doc: fix typo in Japanese release note.

Doc: Mention that double quotes are required in "PGPOOL SHOW" command if the parameter contains uppercase letters.

Fix backend_flag* parameter shown twice while executing "pgpool show all".

There are two entries of "backend_flag" for "ALLOW_TO_FAILOVER" and
"ALWAYS_PRIMARY" in the config variable struct. This is mostly ok but
"pgpool show all" command displayed both backend_flag entries, which
looks redundant. The reason for this is, report_all_variables() shows
grouped variables first then other variables except already shown as
grouped variables.  Unfortunately build_variable groups() is not smart
enough to build grouped variable data: it only registers the first
backend_flag entry and leaves the second entry. Since the second entry
is not a grouped variable, backend_flag is shown firstly as a grouped
variable and then is show as a non grouped variable in
report_all_variables(). To fix this, mark that the second variable is
also a grouped variable (the flag is set by
build_config_variables()). See bug 728 for the report of the problem.

Also add/fix comments.

Doc: add new parameter to recovery_1st_stage_command and recovery_2ndt_stage_command.

Add missing changes in the previous commit.

Add new parameter to recovery_1st_stage_command and recovery_2nd_stage_command.

The new parameter represents the primary (main) server host name which
is defined in backend_hostname parameter. The commands used to obtain
the host name by using hostname command. This is usually fine but in
some systems the host name returned by hostname command is not same as
backend_hostname. This brings problem to detach_false_primary feature.

Note that documentations are not completed in this commit because of
my development environment (Ubuntu 20) issue (openjade crashes). I
will push document changes later using another PC.

Discussion:
https://www.pgpool.net/pipermail/pgpool-hackers/2021-July/003943.html
https://www.pgpool.net/pipermail/pgpool-hackers/2021-July/003974.html

Add release notes.

Doc: Update configuration example "Pgpool-II on Kubernetes".

Doc: add more explanation about backend_application_name.

Allow to log error messages in extended query mode in certain cases.

In the extended query mode, it is possible that error messages are not
logged even after a sync message is received. With this commit
whenever ERROR response is arrived from backend in the extended query
mode, the error message is logged.

Implementing the follow_primary command-locking over the watchdog channel.

Supplementary fix for [pgpool-hackers: 3892] Problem with detach_false_primary..

commit:455f00dd5f5b7b94bd91aa0b6b40aab21dceabb9 fixed a race condition between
detach_false_primary and follow_primary commands. Part of the fix was to make
sure that the detach_false_primary should only be executed on the
leader watchdog node.

The mentioned commit ensures the execution of detach_false_primary on the
watchdog leader by getting the watchdog status from within the main process.
The design is good enough for most cases, but has the potential to fail if
the cluster goes into the election process just after the main process
has read the status.

To fix that, this commit implements the synchronization of follow_primary_command
execution using the distributed locks over the watchdog channel.

The idea is, just before executing the follow_primary during the failover process
we instruct all standby watchdog nodes to acquire a lock on their respective
nodes to block the false primary detection during the period when the
follow_primary is being executed on the leader watchdog node.

Moreover to keep the watchdog process blocked on waiting for the lock the commit
introduced the pending remote lock mechanism, so that remote locks can get
acquired in the background after the completion of the inflight replication checks.

Finally, REQ_DETAIL_CONFIRMED flag is removed from degenerate_backend_set()
request that gets issued to detach the false primary, That means all quorum
and consensus rules must be satisfied for the detach to happen.

Doc: Fix documentation typos.

Doc: Fix documentation typos.

Use milliseconds logging for Pgpool-II and PostgreSQL.

This will make analyzing timing sensitive issues easier. Before we
could not analyze sub-second timing issues.

Add support for log time stamp with milliseconds.

Fix 033.prefer_lower_standbu_delay

Sometimes this test failed due to the waiting time after pcp_reload_conf
was short. I modified the script to confirm the change by PGPOOL SHOW
before running the test.

Fix client side hang when describe message is followed by NoData response.

For certain describe request for a query including "LISTEN", NoData
message can be returned. Since Pgpool-II buffered NoData, clients
could hang waiting for the NoData message in vain. To fix this, do not
buffer NoData message.

Problem reported and patch provided by Daniel van de Giessen.
Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2021-July/003951.html

Fix query cache to not cache SQLValueFunctions (CURRENT_TIME, CURRENT_USER etc.).

Checking SQLValueFunctions was missed whether to be cached or not and
they were regarded as non function objects. As a result they were
cached.

Also add more test cases.