Fix for [pgpool-general: 5315] pg_terminate_backend

Pgpool-II process the pg_terminate_backend by setting the swallow_termination
flag of the backend connection_info referred in the pg_terminate_backend
function, and latter resets that flag when the query execution completes.

The problem with this approach is that if the command complete for
the pg_terminate_backend is received before the connection termination,
This termination is regarded as the backend failure since the
swallow_termination flag was already cleared by the Pgpool-II child after
receiving the query completion message.

The solution is to reset the swallow_termination flag only when the
pg_terminate_query query fails otherwise leave it as it is when the query
is successful, Since, after the termination of the connection the flag
will not matter anyway.

Enhance document.

Add note about %m in failover command. Add indexes to "streaming
replication mode", "master slave mode" and "native replication mode".

Adding the missing ExecStop and ExecReload commands to the systemd
service configuration file.

The patch was contributed by supp_k and enhanced by me.

Fix for 281: "segmentation fault" when execute pcp_attach_node

A DEBUG message was trying to de-reference a NULL value.

Fix load balancing bug in streaming replication mode.

In an explicit transaction, any SELECT will be load balanced until
write query is sent. After writing query is sent, any SELECT should be
sent to the primary node. However if a SELECT is sent before a sync
message is sent, this does not work since the treatment of writing
query is done after ready for query message arrives. Solution is, the
treatment for writing query is done in executing the writing query as
well.

The bug has been there since V3.5.

Fix yet another kind mismatch error in streaming replication mode.

1) Parse "BEGIN" using statement S1, and it is sent to both node 0 and 1.

2) Close S1.

3) Parse SELECT using S1, and it is sent to node 0 (or 1).

4) Bind retrieves info (sent_messages) regarding S1. Since Pgpool-II
   only removes info on S1 when CloseComplete received, Bind decides
   to send bind message to both node 0 & 1 because it was the info
   regarding BEGIN. Node 0 or 1 tries to bind to non existent
   statement S1.

   As a result, something like "failed to read kind from backend.
   kind mismatch among backends. Possible last query was: "BEGIN" kind
   details are: 0[E: prepared statement "S1" does not exist] 1[3]
   check data consistency among db nodes" occurs.

Note that in 3) if other than S1 is used, the problem does not occur.
Solution is, removing S1 when Close message is received. This problem
has been there since 3.5.0 was out.

Fix do_query() hangs after close message.

This is an en-bug in 3.6.1.

If an extend query appears right after a close message, do_query() is
called to check system catalogs, it hangs because it expects to read
pending data. This is caused by being mistakenly set the pending flag
after Close().

Back patch to 3.6-stable and 3.5-stable.

Fixing 0000280: stack smashing detected

It was a buffer overflow in wd_get_cmd function

Fix indentations of query cache documents.

Enhance query cache documents.

Fixing the issue with the watchdog process restart.

When the watchdog process gets abnormally terminated because of some problem
(e.g. Segmentation fault) the new spawned watchdog process fails to start and
produces an error "bind on ... failed with reason: Address already in use".

Reason is the abnormally terminating watchdog process never gets the time to
clean-up the socket it uses for IPC and the new process gets an error because
the socket address is already occupied

Fix is, the Pgpool main process sets the flag in shared memory to mark the
watchdog process was abnormally terminated and at startup when the watchdog
process see that the flag is set, it performs the clean up of the socket file and
also performs the de-escalation (If the watchdog process was crashed when it
was master/coordinator node) if required before initializing itself.

Fix query cache bug reported in pgpool-general-jp:1441.

In streaming replication mode with query cache enabled, SELECT hangs
in the following scenario:

1) a SELECT hits query cache and returns rows from the query cache.
2) following SELECT needs to search meta data and it hangs.

In #1, while returning the cached result, it misses to call
pool_unset_pending_response(), which leave the pending_response flag
be set. In #2, do_query() checks the flag and tries to read pending
response from backend. Since there's no pending data in backend, it
hangs in reading data from backend.

Fix is, just call pool_unset_pending_response() in #1 to reset the
flag.

Bug report and fix provided by Nobuyuki Nagai.

New regression test item (068) added by me.

Remove elog/ereport calls from signal handlers.

elog/ereport calls malloc(), which is not safe to be called inside
signal handlers, per discussion in [pgpool-hackers: 1950].  I ifdef
out them, rather than simply remove them in a hope we someday find a
better solution which make calling the functions inside signal
handlers.

Not that I did not touch exit_handler() of pgpool_main.c because
removing elog/ereport from them loses informative message like
"received smart shutdown request". Pgpool-II main process do not
heavily use malloc(), so the risk is minimum, I guess.

Fix typo in Japanese release notes.

Fix bug failed to create INET domain socket in FreeBSD if listen_addresses = '*'.

per bug202.

Fix for 0000249: watchdog sometimes fails de-escalation.

The logic in pgpool-II main process exit_handler and terminate_all_childrens was
not making sure that pgpool-II main process should only exit after all its
children have exited. And the problem occurs when the main process shutdowns
itself before watchdog and de-escalation child processes.

The solution is to use the waitpid() system call without WNOHANG option.

Fix connection_life_time broken by authentication_timeout

When authentication_timeout is enabled,
connection_life_time could never be expired, because
alarm(0) is called at reading start-up packet.

When there only one connection pool is used, this
problem doesn't occur because the signal handler
for connection_life_time is always set at the end
of the session. However, if more than one connection
pools exist, the handler isn't set but only the time
to colse the connection is calculated.

To fix it, when authentication_timeout is enabled,
save the signal handler for conneciont_life_time
and the remaining time, and undo the handler when
authentication_timeout is disabled.

Fix authentication timeout that can occur right after client connecttions

This is possible when connection_life_time is enabled.

SIGALRM signal is used for both connection_life_time and
authentication_timeout. Usually, SIGALRM is for connection_life_time,
but when the new connection is arrive, read_startup_packet() is called,
and the handler for authentication_timeout is set by pool_signal() and
alarm(authentication_timeout) is called in enable_authentication_timeout().

However, if connection_life_time is expired **between pool_signal() and
alarm()**, authenticate_timeout() will be called when connection_life_time
is expired instead of pool_backend_timer_handler().

To fix this, call alarm() before pool_signal() to prevent the signal
handler from being with wrong timing.

Some changes in release note 3.1-3.6.

Add Japanese release note 3.1-3.6.

Tightening up the watchdog security

Now wd_authkey uses the HMAC SHA-256 hashing.

Add pgpool_adm extension.

Add release 3.1-3.6 release notes.

The contents are not accurate at this moment except 3.6.1. They are
just copy of 3.5.5 release note.

Add 3.5.5 release note.

Update Pgpool-II version to "3.7devel".

Fix occasional segfault when query cache is enabled.

Per bug 263.

Fix packet kind does not match error in extended protocol per bug 231.

According to the bug231, the bug seem to bite you if all of following
conditions are met:

  - Streaming replication mode
  - Load balance node is not node 0
  - Extended protocol is used
  - SELECT is executed, the statement is closed, then a transaction
    command is executed

The sequence of how the problem bites is:

  1) SELECT executes on statement S1 on the load balance node 1
  2) Frontend send Close statement
  3) Pgool-II forward it to backend 1
  4) Frontend sends Parse, Bind, Execute of COMMIT
  5) Pgool-II forward it to backend 0 & 1
  6) Frontend sends sync message
  7) Pgool-II forward it to backend 0 & 1
  8) Backend 0 replies back Parse complete ("1"), while backend 1
     replies back close complete ("3") because of #3.
  9) Kind mismatch occurs

The solution is, in #3, let Pgpool-II wait for response from backend
1, but do not read the response message. Later on Pgpool-II's state
machine will read the response from it before the sync message is sent
in #6. With this, backend 1 will reply back "1" in #8, and the kind
mismatch error does not occur.

Also, fix not calling pool_set_doing_extended_query_message() when
receives Close message.  (I don't know why it was missed).

New regression test "067.bug231" was added.

Enhance documentation.

"Tips for Installation" section added.

Enhance documentation.

Add "Tips for Installation" section added.

Fix a race condition in a signal handler per bug 265.

In child.c there's signal handler which calls elog. Since the signal
handler is not blocked against other signals while processing, deadlock
could occur in the system calls in the pgpool shutdown sequence. To
fix the problem, now the signal handler is blocked by using
POOL_SETMASK.

Ideally we should avoid calling elog in signal handlers though.

Fix wrong minimum configuration value for client_idle_limit_in_recovery.

Per bug #264.

Merge branch 'master' of ssh://git.postgresql.org/pgpool2

Some changes in Makefile.in of doc and doc.ja

Allow to execute "make xslthtml" under doc.ja.

Change pgpool.spec to install sgml docs and man.

Add some necessary file to EXTRA_DIST to build sgml file.

Prepare 3.6.0

Unify the term in the Japanese document

Add some missing documents about load-balancing.

Some untranslated statement are also translated.

Adding AWS watchdog example

Translate AWS watchdog example.

Prepare 3.6RC1

Add Japanese and English release note

some watchdog log message enhancements

Translate the difference of docs made in the commit f6ec43456cf91231d92d34fef4b1b44055bc1180.

Fixing a potential crash in pool_stream functions.

POOL_CONNECTION->con_info should be checked for null value before de-referencing
when read or write fails on backend socket.

Fixing the design of failover command propagation on watchdog cluster

Overhauling the design of how failover, failback and promote node commands are
propagated to the watchdog nodes. Previously the watchdog on pgpool-II node that
needs to perform the node command (failover, failback or promote node) used to
broadcast the failover command to all attached pgpool-II nodes. And this
sometimes makes the synchronization issues, especially when the watchdog cluster
contains a large number of nodes and consequently the failover command sometimes
gets executed by more than one pgpool-II.

Now with this commit all the node commands are forwarded to the
master/coordinator watchdog, which in turn propagates to all standby nodes.
Apart from above the commit also changes the failover command interlocking
mechanism and now only the master/coordinator node can become the lock holder
so the failover commands will only get executed on the master/coordinator node.

Fix the broken log_destination = syslog functionality.

Fixing the logging to the syslog destination, which got broken by the
PGPOOL SET/SHOW command commit, and also enhancing the log_destination
configuration parameter to be assigned with the comma separated list of multiple
destinations for the Pgpool-II log. Now, after this commit log_destination can
be set to any combination of 'syslog' and 'stderr' log destinations.

Fix untranslated sentence.

Adjust RPM installing description to pgpool-II-pg96.

Add RPM installation section.

Enhance the release note.

Add mention that now man pages are generated from SGML documents.

Merge branch 'master' of ssh://git.postgresql.org/pgpool2

specファイルを変更

Prepare 3.6beta2

Fix compile error and add man page

Fix compile error

fix 3.6alpha1, beta1 compile error and  allow "make dist" to include generated man apges.

Allow to create man pages from SGML.

For this purpose, following changes are made:

- Allow to properly create pgpool.xml. For this purpose, add checking
  for xmllint to configure.ac.

- Fix m4/docbook.m4 to not throw an error and properly set
  DOCKBOOKSTYLE variable.  (not sure if this is related to the fix
  though. Anyway, this seems suppress shell errors while configure)

- Allow osx to process UTF-8 documents in doc.ja directory.

- Fix SGML tags in start.sgml and installation.sgml to suppress XML
  validation errors generated by xmllint.

Prepare 3.6beta1

Fix typo in release-3.6.sgml.

Allow to "make dist" to include generated html files.

Prepare 3.6 beta1

add change in doc of 3.6 in pcp_node_info command

Translate examples.sgml into Japanese

Fix typos

Translate client-auth.sgml into Japanese

fix 3.6 release note

fix typo in release note

Translate SGML into Japanese

- failover.sgml
- memcache.sgml
- misc-config.sgml
- online-recovery.sgml
- ssl.sgml
- stream-check.sgml
- watchdog.sgml

Fix typos in Japanese translation of release-3.6.sgml.

So far, up to "Fix for avoiding downtime when
<productname>Pgpool-II</productname> changes require a
restart. (Muhammad Usama)" had been checked.

Merge branch 'master' of ssh://git.postgresql.org/pgpool2

add Japanese 3.6 releasenote

Translate healthcheck.sgml into Japanese

Fix a error of commenting-out

Add missing translation into loadbalance.sgml

Translate loadbalance.sgml into Japanese

Translate connection-pooling.sgml into Japanese

Fix a paramete type

Remove white space at end of line

Translate connection-setting.sgml into Japanes

add Japanese docs for reference

Translate connection-setting.sgml into Japanese

Some types fixes and elaboration are also done.

Adopt the incompatibility with psql of PostgreSQL 9.6.

Since -c option does not imply -X anymore in 9.6, regression fails if
.psqlrc is set, for example "\pset pager" is set.

Do not cancel a query when the query resulted in an error other than in native replication mode.

It was intended to keep the consistency, but there's no point in other
than native replication mode.

Remove obsoleted option "-c".

Also fix typo in the help message.

Minor enhancement on ref/pgpool.sgml.

Translate ref/pgpool.sgml into Japanese.

Also change document id to "Pgpool-II" to not conflict with the id in
ref/pgpool.sgml.

Add pgpool command reference manual.

To my surprise, there was no such a manual in the old HTML docs.

Translate config.sgml into Japanese

Add description about supported platform to intro.sgml.

Update Japanese translation according to the updating of English doc.

Rephrasing a section about start/stop watchdog in advanced.sgml

translate some reference docs into Japanese

Translate advanced.sgml into Japanese.

Also fix typos in the English doc.

Translate runtime.sgml into Japanese

Translate installation.sgml into Japanese

Translate watchdog.sgml into Japanese.

Translate figure title.

Fix typos and add some tags