[pgpool-committers: 7309] pgpool: Doc: fix misleading description about reloading.

Tatsuo Ishii ishii at sraoss.co.jp
Wed Dec 9 11:50:59 JST 2020


Doc: fix misleading description about reloading.

While taking care of this:
https://www.pgpool.net/mantisbt/view.php?id=671

I noticed our documentation gives misleading explanation about pgpool
behavior when pgpool.conf is reloaded:

https://www.pgpool.net/docs/latest/en/html/config-setting.html#CONFIG-SETTING-CONFIGURATION-FILE
The configuration file is reread whenever the main server process
receives a SIGHUP signal; this signal is most easily sent by running
pgpool reload from the command line. The main pgpool process also
propagates this signal to all its child processes, so that existing
sessions also adopt the new values. Some parameters can only be set at
server start; any changes to their entries in the configuration file
will be ignored until the server is restarted. Invalid parameter
settings in the configuration file are likewise ignored (but logged)
during SIGHUP processing.

The reality is, new parameter values will not be reflected to existing
sessions. They will be reflected when new session is
established. Reloading new config values is handled by
check_config_reload() in src/protocol/child.c. It's only called in the
beginning of establishing client sessions. It will not be called while
the session continues. Changing the place to call the function so that
it is called within a session maybe possible. But it will need
careful investigation to not break existing logic depending on
parameters.

Discussion: https://www.pgpool.net/pipermail/pgpool-hackers/2020-December/003856.html

Branch
------
master

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

Modified Files
--------------
doc.ja/src/sgml/config.sgml | 4 ++--
doc/src/sgml/config.sgml    | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)



More information about the pgpool-committers mailing list