From: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
---|---|
To: | Ants Aasma <ants(at)cybertec(dot)at> |
Cc: | Simon Riggs <simon(at)2ndquadrant(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: 9.2.3 crashes during archive recovery |
Date: | 2013-02-21 14:09:02 |
Message-ID: | 51262A7E.7060008@vmware.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 15.02.2013 15:49, Heikki Linnakangas wrote:
> Attached is a patch for git master. The basic idea is to split
> InArchiveRecovery into two variables, InArchiveRecovery and
> ArchiveRecoveryRequested. ArchiveRecoveryRequested is set when
> recovery.conf exists. But if we don't know how far we need to recover,
> we first perform crash recovery with InArchiveRecovery=false. When we
> reach the end of WAL in pg_xlog, InArchiveRecovery is set, and we
> continue with normal archive recovery.
New version of this attached, with a few bugs fixed.
I'm thinking that this should be back-patched to 9.2, but not to earlier
branches. Before 9.2, we don't PANIC at a reference to a non-existent
page until end of recovery, even if we've already reached consistency.
The same basic issue still exists in earlier versions, though: if you
have hot_standby=on, the system will open for read-only queries too
early, before the database is consistent. But this patch is invasive
enough that I'm weary of back-patching it further, when the worst that
can happen is that there's a small window right after startup when you
can see an inconsistent database in hot standby mode. Maybe after we get
some more testing of this in 9.2 and master. Opinions on that?
- Heikki
Attachment | Content-Type | Size |
---|---|---|
crash-recover-before-archive-recovery-2.patch | text/x-diff | 16.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2013-02-21 14:09:48 | Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq |
Previous Message | Peter Eisentraut | 2013-02-21 13:22:16 | Re: Materialized views WIP patch |