Re: [ADMIN] Problems with enums after pg_upgrade

On Tue, Dec 18, 2012 at 10:52:46AM -0500, Andrew Dunstan wrote:
> The translations from oid to label are in pg_enum, but it looks like
> somehow you have lost that mapping. I'm not sure what you've done
> but AFAICT pg_upgrade is doing the right thing.
>
> I just did this (from 9.0 to 9.2) and the pg_upgrade_dump_all.sql
> that is used to create the new catalog has these lines:
>
> -- For binary upgrade, must preserve pg_type oid
> SELECT binary_upgrade.set_next_pg_type_oid('40804'::pg_catalog.oid);
>
>
> -- For binary upgrade, must preserve pg_type array oid
> SELECT
> binary_upgrade.set_next_array_pg_type_oid('40803'::pg_catalog.oid);
>
> CREATE TYPE myenum AS ENUM (
> );
>
> -- For binary upgrade, must preserve pg_enum oids
> SELECT binary_upgrade.set_next_pg_enum_oid('40805'::pg_catalog.oid);
> ALTER TYPE public.myenum ADD VALUE 'foo';
>
> SELECT binary_upgrade.set_next_pg_enum_oid('40806'::pg_catalog.oid);
> ALTER TYPE public.myenum ADD VALUE 'bar';
>
> SELECT binary_upgrade.set_next_pg_enum_oid('40807'::pg_catalog.oid);
> ALTER TYPE public.myenum ADD VALUE 'baz';
>
> and this worked exactly as expected, with a table using this type
> showing the expected values.
>
> Can you produce a test case demonstrating the error?
>
> When you run pg_upgrade, use the -r flag to keep all the
> intermediate files so we can see what's going on.
>
> It's no good dumping the new db looking for these values if they
> have been lost. You would need to have a physical copy of the old db
> and dump that in binary upgrade mode looking for the Oid. If you
> don't have a physical copy of the old db or the intermediate dump
> file pg_upgrade used then recovery is going to be pretty difficult.
> It's not necessarily impossible, but it might involve you getting
> some outside help.

Yes, this matches what I thought too. You see the
binary_upgrade.set_next_pg_enum_oid() calls in pg_dump --binary-upgrade
--schema-only and those set the oid of the newly created enum.

I agree you would need to run this on the _old_ cluster for us to figure
out how it failed.

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

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