Re: [HACKERS] wal_checksum = on (default) | off

From: "Zeugswetter Andreas ADI SD" <ZeugswetterA(at)spardat(dot)at>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "Simon Riggs" <simon(at)2ndquadrant(dot)com>, <pgsql-hackers(at)postgresql(dot)org>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] wal_checksum = on (default) | off
Date: 2007-01-05 16:25:08
Message-ID: E1539E0ED7043848906A8FF995BDA57901A35952@m0143.s-mxs.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


> > 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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dave Page 2007-01-05 17:26:31 -f <output file> option for pg_dumpall
Previous Message Andrew Dunstan 2007-01-05 15:24:13 Re: ideas for auto-processing patches

Browse pgsql-patches by date

  From Date Subject
Next Message Gurjeet Singh 2007-01-05 19:18:47 A patch to pg_regress for Windows port
Previous Message Tom Lane 2007-01-05 14:48:24 Re: Last infomask bit