Re: pg_upgrade if 'postgres' database is dropped
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Bruce Momjian <bruce@momjian.us>
Cc: Magnus Hagander <magnus@hagander.net>, Tom Lane <tgl@sss.pgh.pa.us>, Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>, Stephen Frost <sfrost@snowman.net>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2011-10-28T19:54:16Z
Lists: pgsql-hackers
On Fri, Oct 28, 2011 at 10:18 AM, Bruce Momjian <bruce@momjian.us> wrote: > Bruce Momjian wrote: >> Robert Haas wrote: >> > On Fri, Oct 28, 2011 at 10:07 AM, Bruce Momjian <bruce@momjian.us> wrote: >> > > OK, then the simplest fix, once you modify pg_dumpall, would be to >> > > modify pg_upgrade to remove reference to the postgres database in the >> > > new cluster if it doesn't exist in the old one. ?That would allow >> > > pg_upgrade to maintain a 1-1 matching of databases in the old and new >> > > cluster --- it allows the change to be locallized without affecting much >> > > code. >> > >> > That sounds just fine. +1. >> >> FYI, I don't want to modify pg_dumpall myself because I didn't want to >> have pg_upgrade forcing a pg_dumpall change that applies to >> non-binary-upgrade dumps. pg_dumpall is too important. I am fine if >> someone else does it, though. :-) > > If you want crazy, you could suppress the "\connect postgres" line only > in --binary-upgrade dumps, but that seems to error-prone because then > only --binary-upgrade dumps are exercising that behavior. Also, then it would still be doing something silly in the non --binary-upgrade case. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company