Re: Re: [GENERAL] pg_upgrade from 9.0.7 to 9.1.3: duplicate key pg_authid_oid_index

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bryan Murphy <bmurphy1976(at)gmail(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [GENERAL] pg_upgrade from 9.0.7 to 9.1.3: duplicate key pg_authid_oid_index
Date: 2012-06-13 16:20:21
Message-ID: 20120613162021.GG4419@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers

On Mon, Jun 04, 2012 at 10:16:45AM -0400, Bruce Momjian wrote:
> > I think the checks that are actually needed here are (1) bootstrap
> > superusers are named the same, and (2) there are no roles other than the
> > bootstrap superuser in the new cluster.
>
> You are right that it is more complex than I stated, but given the
> limited feedback I got on the pg_upgrade/plplython, I figured people
> didn't want to hear the details. Here they are:
>
> There are three failure modes for pg_upgrade:
>
> 1. check failure
> 2. schema restore failure
> 3. silent failure/corruption
>
> Of course, the later items are worse than the earlier ones. The
> reporter got a "schema restore failure" while still following the
> pg_upgrade instructions. My initial patch changed that #2 error to a #1
> error. Tom is right that creating users in the new cluster (against
> instructions), can still generate a #2 error if a new/old pg_authid.oid
> match, and they are not the install user, but seeing that is something
> that is against the instructions, I was going to leave that as a #2.

Applied and back-patched to Postgres 9.1.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Misa Simic 2012-06-13 16:21:48 Re: Trying to execute several queries involving temp tables in a PHP script
Previous Message Evan D. Hoffman 2012-06-13 15:19:41 pg_upgrade: "pg_ctl failed to start the new server"

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2012-06-13 16:27:25 Re: [COMMITTERS] pgsql: Mark JSON error detail messages for translation.
Previous Message Andres Freund 2012-06-13 16:16:04 Re: [RFC][PATCH] Logical Replication/BDR prototype and architecture