Re: [DOCS] [HACKERS] Docs for archive_cleanup_command are poor

2010-10-12 Thread Fujii Masao
On Sat, Oct 9, 2010 at 10:04 AM, Brendan Jurd  wrote:
> Hi folks,
>
> I have just set up HS+SR for the first time, and for the most part,
> the docs were excellent.  The one exception for me was the discussion
> of archive_cleanup_command.  This is a pretty important part of
> constructing a healthy standby server, and IMO the docs don't give it
> the treatment it deserves.
>
> Under "25.2.4. Setting Up a Standby Server", we have:
>
> "You can use archive_cleanup_command to prune the archive of files no
> longer needed by the standby."
>
> ... then a few paragraphs later ...
>
> "If you're using a WAL archive, its size can be minimized using the
> archive_cleanup_command  option to remove files that are no longer
> required by the standby server. Note however, that if you're using the
> archive for backup purposes, you need to retain files needed to
> recover from at least the latest base backup, even if they're no
> longer needed by the standby."
>
> So there are a couple of brief mentions of what
> archive_cleanup_command is for, but nothing about how it works, no
> exampes of how to use it, and no links at all.  Contrast how we deal
> with archive_command, restore_command and primary_conninfo.
>
> I'd like to suggest a few ways we could improve on this:
>
> 1. Remove the former paragraph.  It's stranded out there on its own in
> the middle of some unrelated text, and doesn't say anything of
> substance not also said in the latter paragraph.
>
> 2. Include an example archive_cleanup_command in the recovery.conf
> example snippet.
>
> 3. Link to 26.1 which actually explains how a_c_c works.
>
> 4. Mention, and link to, pg_archivecleanup from both 25.2.4 and 26.1.
> This is the utility that most newcomers to WAL archiving will want to
> use, so it's rather weird of us not to advertise it.
>
> I'm willing to write a patch for this, but I thought I'd raise the
> suggestions on-list first, before getting too invested.  So, please
> comment if you have an opinion on this.

Agreed.

And, ISTM that we should mention that we must not just specify
pg_archivecleanup in archive_cleanup_command when there are multiple
standby servers. This is because, in that case, we must calculate
the oldest restart point in those standbys and delete the archived
WAL files according to that point.

Regards,

-- 
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

-- 
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs


[DOCS] Documenting removal of nonnullvalue() and friends

2010-10-12 Thread Josh Kupershmidt
Hi,

It seems that this commit:

commit 2cf8afe5d11377faaf5721b2c2f0089436abf704
Author: Peter Eisentraut 
Date:   Sun Oct 5 17:33:17 2008 +

Remove obsolete internal functions istrue, isfalse, isnottrue, isnotfalse,
nullvalue, nonvalue.  A long time ago, these were used to implement the SQL
constructs IS TRUE, etc.

removed several undocumented functions which probably weren't intended
for wide use. The commit message claims these functions are
"internal", though they were accessible as SQL-language functions.  At
least one person had been using them on 8.3, and was surprised at
seeing them gone with no explanation:
  http://archives.postgresql.org/pgsql-novice/2010-10/msg00084.php

Perhaps a mention of this change could be made in the 8.4 release notes?

Josh

-- 
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs


Re: [DOCS] Documenting removal of nonnullvalue() and friends

2010-10-12 Thread Josh Kupershmidt
On Tue, Oct 12, 2010 at 1:03 PM, Josh Kupershmidt  wrote:
> Perhaps a mention of this change could be made in the 8.4 release notes?

And here's a patch for the 8.4 release notes.

Josh


release-8.4-notes.patch
Description: Binary data

-- 
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs