On Tue, Feb 28, 2012 at 09:45:41PM -0500, Bruce Momjian wrote:
> On Tue, Feb 28, 2012 at 02:15:30PM -0500, Bruce Momjian wrote:
> > On Tue, Feb 28, 2012 at 01:24:45PM -0500, Robert Haas wrote:
> > > > Running this script will delete the old cluster's data files:
> > > > /usr/local/pgdev/pg_upgrade/delete_old_cluster.sh
> > >
> > > I think you should rename the old control file just before the step
> > > that says "linking user relation files". That's the point after which
> > > it becomes unsafe to start the old cluster, right?
> >
> > Yes, it is true that that is the danger point, and also it is much less
> > likely to fail at that point --- it usually happens during the schema
> > creation. I would have to add some more conditional wording without
> > clearly stating if the old suffix is present.
>
> OK, I have implemented both Roberts and Àlvaro's ideas in my patch.
> I only add the .old suffix to pg_controldata when link mode is used, and
> I now do it after the schema has been created (the most common failure
> case for pg_upgrade), and just before we actually link files --- both
> very good ideas.
>
> Patch attached; new pg_upgrade output with link mode below.
Patch applied. I will now work on the change to keep the schema restore
and server logs around in case of a failure.
-- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB
http://enterprisedb.com
+ It's impossible for everything to be true. +