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

From Bruce Momjian
Subject Re: Synchronous Commit Doc Patch
Date
Msg-id 200707170531.l6H5V2P17898@momjian.us
Whole thread Raw
In response to Synchronous Commit Doc Patch  ("Simon Riggs" <simon@2ndquadrant.com>)
List pgsql-patches
bruce wrote:
>
> Uh, we need a context diff for this.

Uh, never mind.  The final version has a context diff.

---------------------------------------------------------------------------



>
> ---------------------------------------------------------------------------
>
> Simon Riggs wrote:
> > 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, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 2: Don't 'kill -9' the postmaster
>
> --
>   Bruce Momjian  <bruce@momjian.us>          http://momjian.us
>   EnterpriseDB                               http://www.enterprisedb.com
>
>   + If your life is a hard drive, Christ can be your backup. +

--
  Bruce Momjian  <bruce@momjian.us>          http://momjian.us
  EnterpriseDB                               http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Async Commit, v21
Next
From: Michael Paesold
Date:
Subject: Re: WIP: rewrite numeric division