public inbox for [email protected]  
help / color / mirror / Atom feed
From: Thom Brown <[email protected]>
To: Heikki Linnakangas <[email protected]>
Cc: Fujii Masao <[email protected]>
Cc: [email protected]
Subject: Re: Confusing link in streaming replication section
Date: Thu, 25 Feb 2010 09:19:43 +0000
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

On 25 February 2010 09:15, Heikki Linnakangas
<[email protected]> wrote:
> Fujii Masao wrote:
>> On Thu, Feb 25, 2010 at 5:03 PM, Thom Brown <[email protected]> wrote:
>>> It looks as if Heikki has already changed the page anyway, which
>>> includes and example of a parameter setting for primary_conninfo and
>>> removes links to the libpq connection stuff.  Problem solved :)
>>
>> Yeah, Heikki removed the link from
>> http://developer.postgresql.org/pgdocs/postgres/streaming-replication.html#STREAMING-REPLICATION-SET...
>>
>> but it still exists in
>> http://developer.postgresql.org/pgdocs/postgres/standby-settings.html
>>
>> So I think that we still need to modify the document about
>> primary_conninfo to suit your need. No?
>
> Yeah, it's not very clear as it is.
>
> How about this:
>
> --- a/doc/src/sgml/recovery-config.sgml
> +++ b/doc/src/sgml/recovery-config.sgml
> @@ -193,7 +193,8 @@ restore_command = 'copy "C:\\server\\archivedir\\%f"
> "%p"'
>         <listitem>
>          <para>
>           Specifies a connection string to be used for the standby server
> -          to connect with the primary. This string is in the same format as
> +          to connect with the primary. This string is in the format
> +          accepted by the libpq <function>PQconnectdb</function> function,
>           described in <xref linkend="libpq-connect">. If any option is
>           unspecified in this string, then the corresponding environment
>           variable (see <xref linkend="libpq-envars">) is checked. If the
>

Yes, that'll do :)

Thanks

Thom



view thread (13+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Confusing link in streaming replication section
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox