From: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | <pgsql-hackers(at)postgresql(dot)org>, <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] wal_checksum = on (default) | off |
Date: | 2007-01-04 16:51:50 |
Message-ID: | 1167929510.20749.183.camel@silverbirch.site |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
On Thu, 2007-01-04 at 11:09 -0500, Tom Lane wrote:
> "Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> > On Thu, 2007-01-04 at 10:00 -0500, Tom Lane wrote:
> >> "Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> >>> Recovery can occur with/without same setting of wal_checksum, to avoid
> >>> complications from crashes immediately after turning GUC on.
> >>
> >> Surely not. Otherwise even the "on" setting is not really a defense.
>
> > Only when the CRC is exactly zero, which happens very very rarely.
>
> "It works most of the time" doesn't exactly satisfy me. What's the
> use-case for changing the variable on the fly anyway? Seems a better
> solution is just to lock down the setting at postmaster start.
That would prevent us from using the secondary checkpoint location, in
the case of a crash effecting the primary checkpoint when it is a
shutdown checkpoint where we changed the setting of wal_checksum. It
seemed safer to allow a very rare error through to the next level of
error checking rather than to close the door so tight that recovery
would not be possible in a very rare case.
If your're good with server start, so am I.
--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | markwkm | 2007-01-04 16:55:17 | Re: 8.3 pending patch queue |
Previous Message | Tom Lane | 2007-01-04 16:44:36 | Re: InitPostgres and flatfiles question |
From | Date | Subject | |
---|---|---|---|
Next Message | Florian Weimer | 2007-01-04 16:58:51 | Re: [HACKERS] wal_checksum = on (default) | off |
Previous Message | Bill Moran | 2007-01-04 16:35:28 | Re: [HACKERS] Patch to log usage of temporary files |