On Mon, 2007-07-23 at 21:11 -0400, Tom Lane wrote:
> Alvaro Herrera <[EMAIL PROTECTED]> writes:
> > What's the thing about doing the flush twice in a couple of comments in
> > calls to XLogBackgroundFlush?  Are they just leftover comments from
> > older code?
> 
> I was wondering that too --- they looked like obsolete comments to me.

True, recent API change meant they were slightly off.

> My current thinking BTW is that trying to make XLogBackgroundFlush serve
> two purposes is counterproductive.  It should be dedicated to use by the
> walwriter only, and the checkpoint case should simply read the async
> commit pointer and call regular XLogFlush with it.

OK

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


---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

               http://archives.postgresql.org

Reply via email to