Re: [COMMITTERS] pgsql: Allow multiple hostaddrs to go with multiple hostnames.

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>
Cc: pgsql-committers <pgsql-committers(at)postgresql(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Allow multiple hostaddrs to go with multiple hostnames.
Date: 2017-07-10 10:27:40
Message-ID: CAD21AoBCwcTNMdrVWq8T0hoOs2mWSYq9PRJ_fr6SH8HdO+m=0g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Mon, Jul 10, 2017 at 6:30 PM, Heikki Linnakangas
<heikki(dot)linnakangas(at)iki(dot)fi> wrote:
> Allow multiple hostaddrs to go with multiple hostnames.
>
> Also fix two other issues, while we're at it:
>
> * In error message on connection failure, if multiple network addresses
> were given as the host option, as in "host=127.0.0.1,127.0.0.2", the
> error message printed the address twice.
>
> * If there were many more ports than hostnames, the error message would
> always claim that there was one port too many, even if there was more than
> one. For example, if you gave 2 hostnames and 5 ports, the error message
> claimed that you gave 2 hostnames and 3 ports.
>
> Discussion: https://www.postgresql.org/message-id/10badbc6-4d5a-a769-623a-f7ada43e14dd@iki.fi
>
> Branch
> ------
> master
>
> Details
> -------
> https://git.postgresql.org/pg/commitdiff/7b02ba62e9ffad5b14c24756a0c2aeae839c9d05
>
> Modified Files
> --------------
> doc/src/sgml/libpq.sgml | 55 +++++++-
> src/interfaces/libpq/fe-connect.c | 258 ++++++++++++++++++++++++--------------
> src/interfaces/libpq/libpq-int.h | 3 +-
> 3 files changed, 219 insertions(+), 97 deletions(-)
>
>
> --
> Sent via pgsql-committers mailing list (pgsql-committers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-committers

Hi,

This commit seems be cause of the documentation compilation error. I
think </sect3> is missing.

$ make -C doc/src/sgml/
make: Entering directory `/home/masahiko/pgsql/source/postgresql/doc/src/sgml'
osx -wall -wno-unused-param -wno-empty -wfully-tagged -D . -D . -x
lower postgres.sgml >postgres.xml.tmp
osx:libpq.sgml:926:9:E: end tag for "SECT3" omitted, but OMITTAG NO
was specified
osx:libpq.sgml:891:3: start tag was here
make: *** [postgres.xml] Error 1
make: Leaving directory `/home/masahiko/pgsql/source/postgresql/doc/src/sgml'

Attached small patch fixes this.

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

Attachment Content-Type Size
fix_doc.patch text/x-patch 396 bytes

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Gierth 2017-07-10 10:44:06 pgsql: Fix COPY's handling of transition tables with indexes.
Previous Message Heikki Linnakangas 2017-07-10 09:30:43 pgsql: Allow multiple hostaddrs to go with multiple hostnames.

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2017-07-10 10:27:48 Re: Partition-wise join for join between (declaratively) partitioned tables
Previous Message Andrew Gierth 2017-07-10 10:10:12 Re: COPY vs. transition tables