Re: confusing archive_command example

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Greg Smith <greg(at)2ndquadrant(dot)com>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Josh Kupershmidt <schmiddy(at)gmail(dot)com>, pgsql-docs(at)postgresql(dot)org
Subject: Re: confusing archive_command example
Date: 2010-04-12 22:05:56
Message-ID: 201004122205.o3CM5vc27021@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

Greg Smith wrote:
> The third issue here, not mentioned yet, is that there's also an example
> of setting up archive_command in its GUC documentation:
>
> http://developer.postgresql.org/pgdocs/postgres/runtime-config-wal.html
>
> This has drifted out of sync with the section being updated here this
> week--its archive_command example isn't even using "-i". I think the
> right thing to do here is to migrate the Windows example out of 18.5.3
> from 24.3.1, then replace the existing examples in 18.5.3 to instead
> suggest 24.3.1 has them. Given that all of this documentation giving
> examples for how to correctly configure that parameter, like this cp
> trivia, only appears in the PITR docs section, I don't see any reason to
> duplicate that in the GUC area. Having the second, minimally documented
> version of those in 18.5.3 just encourages people to use what we know is
> a bad practice by providing them as examples.

Agreed. The attached applied patch removes the example from the
configure section.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

Attachment Content-Type Size
/rtmp/diff text/x-diff 2.1 KB

In response to

Browse pgsql-docs by date

  From Date Subject
Next Message Fujii Masao 2010-04-13 00:31:43 typo in config.sgml
Previous Message Alvaro Herrera 2010-04-08 17:48:09 Re: PDF build issue with 9.0 Alpha5