[pgpool-hackers: 691] Re: [pgpool-committers: 2326] pgpool: Reorganize port numbers assigned in pgool_setup.

[Tatsuo Ishii]

Hi pgpool hackers,

I have made following changes to pgpool_setup (and regression test to
meet the pgpool_setup change). The idea behind this is to fix the port
number assigned to pgpool and pcp. I was too often confused by the
pgpool port number, sometimes accidentally connect to PostgreSQL,
rather than pgpool and wasted my time. Now I can always use port 11000
to connect to pgpool and 11001 for pcp(unless you specify base port
number). I hope this change is good for you as well.

This change applied to master and 3.4-stable branch. I'm a little bit
hesitate to apply to 3.3-stable since this is not a trivial change.

Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp

> Reorganize port numbers assigned in pgool_setup.
> 
> Assign base port (11000 by default) to pgpool, base port + 1 to pcp,
> port + 2 to PostgreSQL node0, port + 3 to PostgreSQL node1 and so on.
> This assigns fixed port numbers to pgpool and pcp port, which makes
> pgpool_setup user's life a little bit easier, because they don't need
> to check README.port to make sure the pgpool and pcp port any more.
> 
> Some of regression tests uses immediate numeric port numbers and
> adjusted to meet the change.
> 
> Branch
> ------
> master
> 
> Details
> -------
> http://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=0920faac9daeb4ccc280a5a17bb443acd6e498c9
> 
> Modified Files
> --------------
> src/test/pgpool_setup                              |   26 ++++++++++++--------
> .../tests/002.native_replication/test.sh           |    4 +--
> src/test/regression/tests/003.failover/expected.r  |    4 +--
> src/test/regression/tests/003.failover/expected.s  |    4 +--
> src/test/regression/tests/004.watchdog/master.conf |    2 +-
> .../regression/tests/004.watchdog/standby.conf     |    6 ++---
> src/test/regression/tests/054.postgres_fdw/test.sh |   10 ++++----
> .../regression/tests/055.backend_all_down/test.sh  |    2 +-
> 8 files changed, 32 insertions(+), 26 deletions(-)