On 10 January 2013 06:06, Jeff Davis <pgsql@j-davis.com> wrote:
> The checksums patch also introduces another behavior into
> SetBufferCommitInfoNeedsSave, which is to write an XLOG_HINT WAL record
> if checksums are enabled (to avoid torn page hazards). That's only
> necessary for changes where the caller does not write WAL itself and
> doesn't bump the LSN of the data page. (There's a reason the caller
> can't easily write the XLOG_HINT WAL itself.) So, we could introduce
> another flag "needsWAL" that would control whether we write the
> XLOG_HINT WAL or not (only applies with checksums on, of course).
That wouldn't work because it can't know the exact answer to that, but
the way the patch does this is already correct.
XLOG_HINT_WAL doesn't always write a WAL record, it only does it when
necessary. See XLogInsert()
Didn't fully understand other comments. Do we we need an answer now?
My head is somewhere else.
-- Simon Riggs http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training & Services