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: Michael Fuhr <mike(at)fuhr(dot)org>,PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Patch for disaster recovery
Date: 2005-02-20 18:36:21
Message-ID: 200502201836.j1KIaLl10543@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:
> > Tom Lane wrote:
> >> Not sure where this leads to, but it's not leading to an undocumented
> >> one-line hack in tqual.c, and definitely not *that* one-line hack.
> 
> > Sorry, here is the proper change I just applied:
> 
> > /* This is to be used only for disaster recovery and requires serious analysis. */
> > #ifndef MAKE_ALL_TUPLES_VISIBLE
> >     return false;
> > #else
> >     return true;
> > #endif
> 
> AFAICS this has no value whatsoever.  Assuming that someone has a
> disaster recovery problem on their hands, how likely is it that they
> will know that that code is there, or be able to turn it on (most
> users don't compile from source anymore), or be able to use it
> effectively, given the complete lack of documentation?  As is, this
> is of value only to someone familiar with the code, and such a someone
> could go in and modify the logic for themselves just as easily as
> turn on a #define.
> 
> I think the only real effect of this patch will be to confuse people
> who are reading the source code.  tqual.c is already complicated and
> fragile enough --- it doesn't need conditionally compiled "features"
> that we can't even explain the use of.

I need a note somewhere to remember where to tell people to modify the
code to recovery something.  Do you have a better idea?  You want just a
comment rather than a define?

-- 
  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: Magnus HaganderDate: 2005-02-20 18:56:10
Subject: pg_ctl reference page
Previous:From: Tom LaneDate: 2005-02-20 18:20:35
Subject: Re: Patch for disaster recovery

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