pg.ddx.io  pgsql-patches@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Gregory Stark <stark@enterprisedb.com>
To: Simon Riggs <simon@2ndQuadrant.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>
Cc: ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>
Cc: pgsql-hackers@postgresql.org
Subject: Re: rmgr hooks and contrib/rmgr_hook
Date: Mon, 15 Sep 2008 16:43:54 +0100
Message-ID: <87k5ddmn2d.fsf@oxford.xeocode.com> (raw)
In-Reply-To: <1221490804.3913.1406.camel@ebony.2ndQuadrant>
References: <20080901121833.4A18.52131E4D@oss.ntt.co.jp>
	<1220345064.4371.275.camel@ebony.2ndQuadrant>
	<20080902174613.76B2.52131E4D@oss.ntt.co.jp>
	<1220348980.4371.316.camel@ebony.2ndQuadrant>
	<14451.1220369969@sss.pgh.pa.us>
	<1220372673.4371.465.camel@ebony.2ndQuadrant>
	<48CB72C8.1070700@enterprisedb.com>
	<1221470800.3913.1229.camel@ebony.2ndQuadrant>
	<87sks1oi4o.fsf@oxford.xeocode.com>
	<24006.1221483138@sss.pgh.pa.us>
	<1221485063.3913.1350.camel@ebony.2ndQuadrant>
	<25319.1221487473@sss.pgh.pa.us>
	<1221490804.3913.1406.camel@ebony.2ndQuadrant>

Simon Riggs <simon@2ndQuadrant.com> writes:

> Bottom line is that any backup of Postgres needs to include plugin
> directories, otherwise parts of the application could stop working
> following restore. This patch doesn't change that.

No, backups of executables are normally not the same backups as the data and
in many cases -- minor upgrades for example -- cannot be.

> * add the rmgr bms to the long header of each WAL file
>
> * change !RmgrIdIsValid() so that it causes FATAL by default. This then
> allows people to correct a mistake and retry. We provide an option to
> treat such errors as corrupt data and assume we have reached the end of
> WAL.

I'm not sure but I think this just begs the question. The problem is to ensure
that the rmgrid means the same thing on the restoring database as it does on
the original database, or at least a compatible version. I think this would
mean having a long text description and version number to compare.

And as Tom points out startup isn't often enough. Would WAL headers even be
often enough? We would have to ensure there was never two versions of the
plugin in the same WAL file.

-- 
  Gregory Stark
  EnterpriseDB          http://www.enterprisedb.com
  Ask me about EnterpriseDB's On-Demand Production Tuning



view thread (23+ messages)  latest in thread

Message-ID: <87k5ddmn2d.fsf@oxford.xeocode.com>
Permalink:  ../87k5ddmn2d.fsf@oxford.xeocode.com/
Also on:    postgresql.org/message-id/87k5ddmn2d.fsf@oxford.xeocode.com

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-patches@postgresql.org
  Cc: stark@enterprisedb.com, simon@2ndQuadrant.com, tgl@sss.pgh.pa.us, heikki.linnakangas@enterprisedb.com, itagaki.takahiro@oss.ntt.co.jp, pgsql-hackers@postgresql.org
  Subject: Re: rmgr hooks and contrib/rmgr_hook
  In-Reply-To: <87k5ddmn2d.fsf@oxford.xeocode.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox