Re: Synchronous Commit Doc Patch - Mailing list pgsql-patches

From Simon Riggs
Subject Re: Synchronous Commit Doc Patch
Date
Msg-id 1184335180.4512.156.camel@ebony.site
Whole thread Raw
In response to Synchronous Commit Doc Patch  ("Simon Riggs" <simon@2ndquadrant.com>)
Responses Re: Synchronous Commit Doc Patch  ("Simon Riggs" <simon@2ndquadrant.com>)
Re: Synchronous Commit Doc Patch  (Bruce Momjian <bruce@momjian.us>)
Re: Synchronous Commit Doc Patch  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
On Fri, 2007-07-13 at 15:01 +0200, Reviewer wrote:

> I just read your patch in order to understand a little bit what will
> happen in the next release.

Thanks for the review. v2 attached.

> About the guc variable wal_writer_delay, you say at the end:
> "This parameter can only
> +         be set in the postgresql.conf file or on the server command line."
>
> Well, I don't really understand if this parameter is set only at server
> startup or if it is possible to modify the postgresql.conf file and then
> reload the conf-files.

A common confusion. I'm not personally in favour of using that phrase in
the docs, though it is the standard description of when to set this type
of parameter, so I have followed the convention.

> Also, in the part about synchronous commit and fsync=off, there's just a
> missing verb, I think you should insist between the two possibilities :
> system or database server crash, maybe by using some bold.

I've reworded that part. Thanks for spotting this.

>  From a user point of view, I think the documentation is clear enough,
> but I admit beeing a bit lost in the real end of the explanation, the
> part about the transaction status hint bits but I think it's not that
> important. I think I understood synchronous and asynchronous commits.

I've added to and reworked that a bit also.

--
  Simon Riggs
  EnterpriseDB  http://www.enterprisedb.com

Attachment

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgstat_drop_relation bugfix
Next
From: "Simon Riggs"
Date:
Subject: Re: Synchronous Commit Doc Patch