Re: [PATCHES] Restartable Recovery - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: [PATCHES] Restartable Recovery
Date
Msg-id 1153079785.2654.266.camel@localhost.localdomain
Whole thread Raw
In response to Re: [PATCHES] Restartable Recovery  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCHES] Restartable Recovery
List pgsql-hackers
On Sun, 2006-07-16 at 15:33 -0400, Tom Lane wrote:
> Simon Riggs <simon@2ndquadrant.com> writes:
> > On Sun, 2006-07-16 at 12:40 -0400, Tom Lane wrote:
> >> A compromise that might be good enough is to add an rmgr routine defined
> >> as "bool is_idle(void)" that tests whether the rmgr has any open state
> >> to worry about.  Then, recovery checkpoints are done only if all rmgrs
> >> say they are idle.  
> 
> > Perhaps that should be extended to say whether there are any
> > non-idempotent changes made in the last checkpoint period. That might
> > cover a wider set of potential actions.
> 
> Perhaps best to call it safe_to_checkpoint(), and not pre-judge what
> reasons the rmgr might have for not wanting to restart here.

You read my mind.

> If we are only going to do a recovery checkpoint at every Nth checkpoint
> record, then occasionally having to skip one seems no big problem ---
> just do it at the first subsequent record that is safe.

Got it.

--  Simon Riggs EnterpriseDB          http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCHES] Restartable Recovery
Next
From: Andrew Dunstan
Date:
Subject: Re: Windows buildfarm support, or lack of it