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