Re: pgsql: Unite ReadBufferWithFork, ReadBufferWithStrategy, and - Mailing list pgsql-committers

From Simon Riggs
Subject Re: pgsql: Unite ReadBufferWithFork, ReadBufferWithStrategy, and
Date
Msg-id 1225485460.3971.618.camel@ebony.2ndQuadrant
Whole thread Raw
In response to pgsql: Unite ReadBufferWithFork, ReadBufferWithStrategy, and  (heikki@postgresql.org (Heikki Linnakangas))
Responses Re: pgsql: Unite ReadBufferWithFork, ReadBufferWithStrategy, and  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
List pgsql-committers
On Fri, 2008-10-31 at 15:05 +0000, Heikki Linnakangas wrote:
> Log Message:
> -----------
> Unite ReadBufferWithFork, ReadBufferWithStrategy, and ZeroOrReadBuffer
> functions into one ReadBufferExtended function, that takes the strategy
> and mode as argument. There's three modes, RBM_NORMAL which is the default
> used by plain ReadBuffer(), RBM_ZERO, which replaces ZeroOrReadBuffer, and
> a new mode RBM_ZERO_ON_ERROR, which allows callers to read corrupt pages
> without throwing an error. The FSM needs the new mode to recover from
> corrupt pages, which could happend if we crash after extending an FSM file,
> and the new page is "torn".

I thought you were adding the "read buffer only if in cache" option
also?

--
 Simon Riggs           www.2ndQuadrant.com
 PostgreSQL Training, Services and Support


pgsql-committers by date:

Previous
From: heikki@postgresql.org (Heikki Linnakangas)
Date:
Subject: pgsql: Update FSM on WAL replay.
Next
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Simplify ExecutorRun's API and save some trivial number of cycles