Re: Patch for disaster recovery

Bruce Momjian <pgman@candle.pha.pa.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.

            regards, tom lane