[pgpool-hackers: 2270] Re: [pgpool-committers: 3978] pgpool: Implement infrastructure for supporting per node health check p

[Muhammad Usama]

On Thu, Apr 20, 2017 at 5:49 AM, Tatsuo Ishii <ishii at sraoss.co.jp> wrote:

> In this implementation, I have removed "failback to template1 database
> if postgres database does not exist while doing health check"
> feature. The reasons are:
>
> - It is pretty rare that the database postgres does not exist.
> - Even if the case, user can specify alternate database.
> - The failback feature makes the code complex and reduces maintainability
>   and reliability.
>
> For now, the code works as follows:
>
> - If health check database is specified, use it.
>
> - If health check database is not specified (i.e. empty), use postgres
>   database instead.
>
> If there's no objection, I would like to go forward this way (no
> failback feature).
>

I think this totally make sense to remove the tailback to template1
database for health check, Since the health check database is configurable
so there is no point in failing back to template1 db.

Kind Regards
Muhammad Usama



>
> Best regards,
> --
> Tatsuo Ishii
> SRA OSS, Inc. Japan
> English: http://www.sraoss.co.jp/index_en.php
> Japanese:http://www.sraoss.co.jp
>
> > Implement infrastructure for supporting per node health check parameters.
> >
> > Now pgpool main process forks health check process for each DB node.
> > Also autotools is now updated from 1.14 to 1.15.
> >
> > TODO:
> > - Implement per node health check parameters.
> >
> > Branch
> > ------
> > master
> >
> > Details
> > -------
> > https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=
> 127e64fa52fe2ad2337eea5d55a0b8104f0a4df6
> >
> > Modified Files
> > --------------
> > Makefile.in                         |  49 +++--
> > aclocal.m4                          |  71 ++++----
> > configure                           |  85 +++++----
> > doc.ja/Makefile.in                  |  24 ++-
> > doc.ja/src/Makefile.in              |  24 ++-
> > doc.ja/src/sgml/Makefile.in         |  24 ++-
> > doc/Makefile.in                     |  24 ++-
> > doc/src/Makefile.in                 |  24 ++-
> > doc/src/sgml/Makefile.in            |  24 ++-
> > src/Makefile.am                     |   1 +
> > src/Makefile.in                     |  34 +++-
> > src/include/Makefile.in             |  25 ++-
> > src/include/pool.h                  |   4 +-
> > src/libs/Makefile.in                |  24 ++-
> > src/libs/pcp/Makefile.in            |  27 ++-
> > src/main/health_check.c             | 352 ++++++++++++++++++++++++++++++
> ++++++
> > src/main/pgpool_main.c              | 247 +++++--------------------
> > src/parser/Makefile.in              |  26 ++-
> > src/protocol/pool_connection_pool.c |  17 +-
> > src/tools/Makefile.in               |  25 ++-
> > src/tools/pcp/Makefile.in           |  25 ++-
> > src/tools/pgmd5/Makefile.in         |  26 ++-
> > src/watchdog/Makefile.in            |  25 ++-
> > 23 files changed, 815 insertions(+), 392 deletions(-)
> >
> _______________________________________________
> pgpool-hackers mailing list
> pgpool-hackers at pgpool.net
> http://www.pgpool.net/mailman/listinfo/pgpool-hackers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.sraoss.jp/pipermail/pgpool-hackers/attachments/20170420/ff0565fb/attachment.html>