Re: [PATCHES] wal_checksum = on (default) | off - Mailing list pgsql-hackers

From Zeugswetter Andreas ADI SD
Subject Re: [PATCHES] wal_checksum = on (default) | off
Date
Msg-id E1539E0ED7043848906A8FF995BDA57901A35952@m0143.s-mxs.net
Whole thread Raw
In response to Re: [PATCHES] wal_checksum = on (default) | off  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> > Ok, so when you need CRC's on a replicate (but not on the master)
you
> > turn it
> > off during standby replay, but turn it on when you start the
replicate
> > for normal operation.
>
> Thought: even when it's off, the CRC had better be computed for
> shutdown-checkpoint records.  Else there's no way to turn it on even
> with a postmaster restart --- unless we accept the idea of poking a
hole
> in the normal mode.  (Which I still dislike, and even more so if the
> special value is zero.  Almost any other value would be safer than
zero.)
>
> On the whole, though, I still don't want to put this in.  I don't
think
> Simon has thought it through sufficiently,

Well, the part that we do not really want a special value (at least not
0)
is new, and makes things a bit more complicated.

> and we haven't even seen any demonstration of a big speedup.

Yes, iirc the demonstration was with the 64 bit crc instead of the
sufficient
32-bit (or a bad crc compiler optimization?).
But I do think it can be shown to provide significant speedup
(at least peak burst performance).

Especially on target hardware WAL write IO is extremely fast
(since it is write cached), so the CPU should show.

Andreas

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: ideas for auto-processing patches
Next
From: Dave Page
Date:
Subject: -f option for pg_dumpall