On Thu, Jan 4, 2018 at 11:53 AM, Michael Paquier <michael.paqu...@gmail.com> wrote:
> On Wed, Jan 03, 2018 at 06:48:07PM +1100, Haribabu Kommi wrote: > > On Wed, Jan 3, 2018 at 12:25 PM, Haribabu Kommi < > kommi.harib...@gmail.com> > > Last patch has undefined symbol, corrected patch attached. > Thanks for the review. + memset(walrcv->host, 0, NAMEDATALEN); > + if (host) > + strlcpy((char *) walrcv->host, host, NAMEDATALEN); > + > + memset(walrcv->hostaddr, 0, NAMEDATALEN); > + if (hostaddr) > + strlcpy((char *) walrcv->hostaddr, hostaddr, NAMEDATALEN); > You need to use NI_MAXHOST for both things here. > Corrected. > + <row> > + <entry><structfield>remote_hostname</structfield></entry> > + <entry><type>text</type></entry> > + <entry>Host name of the PostgreSQL instance this WAL receiver is > connected to</entry> > + </row> > PostgreSQL is usualy referred to with the <productname> markup. Those > should be split on multiple lines. The doc changes are nits though. > updated the documentation with markups. I have done some testing with this patch with primary_conninfo using > multiple values of host and port, and the correct values are being > reported, which is a nice feature. > Thanks for testing. updated patch attached. Regards, Hari Babu Fujitsu Australia
pg_stat_wal_receiver-to-display-connected-host_v4.patch
Description: Binary data