Re: Bug in pg_dump/restore -o - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: Bug in pg_dump/restore -o
Date
Msg-id 200201181702.g0IH2Vt20652@candle.pha.pa.us
Whole thread Raw
In response to Re: Bug in pg_dump/restore -o  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Bug in pg_dump/restore -o  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > Philip Warner wrote:
> >> Here's a patch that seems to do the trick. It may seem large, but it is
> >> primarily a reorganization of existing code in pg_backup_db.c to make the
> >> code a little clearer and to facilitate the swapping around of some loops.
> 
> > I hate to say it but I think this is too risky for 7.2.
> 
> It does seem a rather large diff ... however, if I understand it right,
> what Philip has done here is to hack pg_restore so that it will do the
> right thing with an unmodified 7.1 dump file.  This is much superior to
> hacking pg_dump to change the emitted dump data.  Weren't you telling me
> that we might need a solution for someone who had already done pg_dump
> -o -Ft and then wiped their 7.1 database?
> 
> I think we should scrutinize this as carefully as we can, but apply it
> if at all possible.

Interesting solution.  I defer to your opinion on this.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: "Dwayne Miller"
Date:
Subject: 7.2b5 notes
Next
From: Liam Stewart
Date:
Subject: Re: [PATCHES] guc