Skip site navigation (1) Skip section navigation (2)

Re: Patch for disaster recovery

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Patch for disaster recovery
Date: 2005-02-20 14:43:11
Message-ID: 200502201443.j1KEhB400383@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-patches
Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > + /* This is to be used only for disaster recovery and requires serious analysis. */
> > + #ifdef MAKE_ALL_TUPLES_VISIBLE
> > + 	return true;
> > + #endif
> 
> If this requires serious analysis, why did you install the patch without
> any public discussion?
> 
> I cannot imagine any situation where this wouldn't break things beyond
> belief, and so I request that you revert the patch until you've provided
> a real example in which it would help.

We regularly have people on IRC who delete data and then want to recover
it.  By having the define it makes it easier for us to help them without
them having to add actual C code.

Does that make sense?

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

In response to

Responses

pgsql-patches by date

Next:From: Bruce MomjianDate: 2005-02-20 14:45:16
Subject: Re: Patch for disaster recovery
Previous:From: Michael FuhrDate: 2005-02-20 08:35:25
Subject: Re: Patch for disaster recovery

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group