Subject: Re: [COMMITTERS] pgsql: Move documentation of all recovery.conf
	option to a new chapter.
From: Simon Riggs <simon@2ndQuadrant.com>
To: Heikki Linnakangas <heikki@postgresql.org>
Cc: pgsql-hackers@postgresql.org
In-Reply-To: <20100222114730.ADEC37541D0@cvs.postgresql.org>
References: <20100222114730.ADEC37541D0@cvs.postgresql.org>
Content-Type: text/plain
Date: Tue, 23 Feb 2010 23:24:06 +0000
Message-Id: <1266967446.3752.4843.camel@ebony>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit

On Mon, 2010-02-22 at 11:47 +0000, Heikki Linnakangas wrote:
> Log Message:
> -----------
> Move documentation of all recovery.conf option to a new chapter.
> They used to be scattered between the "backup and restore" and "streaming
> replication" chapters.

It's just taken me 15 minutes to locate the settings for
primary_conninfo to better understand Stefan's recent post.

The commit referred to here is an extremely bad change.

If you intended this to be a heading within the "High Availability"
chapter then I would agree. This is what I thought you had done.

Having Streaming Rep described in HA and then describing the parameters
that make it work in a separate chapter is ridiculous. Plus, wherever
they are, they need cross references between them.

This is the last straw for me. The Streaming Rep docs are now truly
appalling. They were thin before, but tieing them in knots and splitting
them into disconnected pieces is just too much. If I can't find the damn
things, we've gone too far.

SR is a big feature and deserves proper docs.

-- 
 Simon Riggs           www.2ndQuadrant.com