* Tom Lane ([email protected]) wrote: > Stephen Frost <[email protected]> writes: > > * Matt Landry ([email protected]) wrote: > >> postgres=# select datname, datallowconn from pg_database ; > >> datname | datallowconn > >> -----------+-------------- > >> template1 | t > >> template0 | t > >> postgres | t > >> reporting | t > >> (4 rows) > > > Right, as I mentioned, template0 shouldn't have datallowconn as 'true'. > > That's why it's being included in the pg_dumpall. > > Perhaps pg_upgrade should deliberately ignore template0 regardless of > datallowconn? And/or we should hard-wire that into pg_dumpall?
My thinking would be that pg_dumpall should be hard-wired for template0
(just like it is for template1..) and that we should *not* be excluding
databases that are marked as datallowconn = false.. That said, it's not
clear to me what to do there instead. Maybe throw an error or a
warning? The point of pg_dumpall is to dump *all* the databases and at
least the manpage doesn't appear to say anything about "but ignores
databases with datallowconn = false".
> I feel no compulsion whatsoever to preserve any user-initiated changes
> in template0 across an upgrade.
I agree with this, with regard to template0.
Thanks,
Stephen
signature.asc
Description: Digital signature
