Thread: pg_upgrade relation OID mismatches

pg_upgrade relation OID mismatches

From
Peter Eisentraut
Date:
I thought these were fixed a while ago, but I'm still seeing these when
upgrading from master to self (using testing script sent in a while
ago).   This is completely reproducible.  What's happening?

...
Restoring user relation files /home/peter/devel/postgresql/git/postgresql/contrib/pg_upgra
Mismatch of relation OID in database "regression": old OID 16701, new OID 16689
Failure, exiting




Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Peter Eisentraut wrote:
> I thought these were fixed a while ago, but I'm still seeing these when
> upgrading from master to self (using testing script sent in a while
> ago).   This is completely reproducible.  What's happening?
> 
> ...
> Restoring user relation files
>   /home/peter/devel/postgresql/git/postgresql/contrib/pg_upgra
> Mismatch of relation OID in database "regression": old OID 16701, new OID 16689
> Failure, exiting

Yes, I certainly thought they were all addressed.  What object is 16701
in the old database?  Anything unusual about it?  This is saying the
relation oid was not preserved.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: pg_upgrade relation OID mismatches

From
Peter Eisentraut
Date:
On tis, 2011-11-22 at 15:42 -0500, Bruce Momjian wrote:
> Peter Eisentraut wrote:
> > I thought these were fixed a while ago, but I'm still seeing these when
> > upgrading from master to self (using testing script sent in a while
> > ago).   This is completely reproducible.  What's happening?
> > 
> > ...
> > Restoring user relation files
> >   /home/peter/devel/postgresql/git/postgresql/contrib/pg_upgra
> > Mismatch of relation OID in database "regression": old OID 16701, new OID 16689
> > Failure, exiting
> 
> Yes, I certainly thought they were all addressed.  What object is 16701
> in the old database?  Anything unusual about it?  This is saying the
> relation oid was not preserved.

It came in with the range types feature:

+ psql -d regression -x -c 'select * from pg_class where oid = 16701'
-[ RECORD 1 ]--+----------------
relname        | test_range_gist
relnamespace   | 2200
reltype        | 16703
reloftype      | 0
relowner       | 10
relam          | 0
relfilenode    | 16701
reltablespace  | 0
relpages       | 33
reltuples      | 6200
relallvisible  | 33
reltoastrelid  | 16704
reltoastidxid  | 0
relhasindex    | t
relisshared    | f
relpersistence | p
relkind        | r
relnatts       | 1
relchecks      | 0
relhasoids     | f
relhaspkey     | f
relhasrules    | f
relhastriggers | f
relhassubclass | f
relfrozenxid   | 1627
relacl         | 
reloptions     | 




Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Peter Eisentraut wrote:
> On tis, 2011-11-22 at 15:42 -0500, Bruce Momjian wrote:
> > Peter Eisentraut wrote:
> > > I thought these were fixed a while ago, but I'm still seeing these when
> > > upgrading from master to self (using testing script sent in a while
> > > ago).   This is completely reproducible.  What's happening?
> > > 
> > > ...
> > > Restoring user relation files
> > >   /home/peter/devel/postgresql/git/postgresql/contrib/pg_upgra
> > > Mismatch of relation OID in database "regression": old OID 16701, new OID 16689
> > > Failure, exiting
> > 
> > Yes, I certainly thought they were all addressed.  What object is 16701
> > in the old database?  Anything unusual about it?  This is saying the
> > relation oid was not preserved.
> 
> It came in with the range types feature:
> 
> + psql -d regression -x -c 'select * from pg_class where oid = 16701'
> -[ RECORD 1 ]--+----------------
> relname        | test_range_gist
> relnamespace   | 2200
> reltype        | 16703
> reloftype      | 0
> relowner       | 10
> relam          | 0
> relfilenode    | 16701
> reltablespace  | 0
> relpages       | 33
> reltuples      | 6200
> relallvisible  | 33
> reltoastrelid  | 16704
> reltoastidxid  | 0
> relhasindex    | t
> relisshared    | f
> relpersistence | p
> relkind        | r
> relnatts       | 1
> relchecks      | 0
> relhasoids     | f
> relhaspkey     | f
> relhasrules    | f
> relhastriggers | f
> relhassubclass | f
> relfrozenxid   | 1627
> relacl         | 
> reloptions     | 

OK, that is a heap table.  My only guess is that the heap is being
created without binary_upgrade_next_heap_pg_class_oid being set.
Looking at the code, I can't see how the heap could be created without
this happening.  Another idea is that pg_dumpall isn't output the proper
value, but again, how is this data type different from the others.

I will try to research this further.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Bruce Momjian wrote:
> OK, that is a heap table.  My only guess is that the heap is being
> created without binary_upgrade_next_heap_pg_class_oid being set.
> Looking at the code, I can't see how the heap could be created without
> this happening.  Another idea is that pg_dumpall isn't output the proper
> value, but again, how is this data type different from the others.

I have reproduced the failure and found it was code I added to pg_dump
back in 9.0.  The code didn't set the index oid for exclusion constraint
indexes.  Once these were added to the regression tests for range types
recently, pg_upgrade threw an error.

My assumption is that anyone trying to use an exclusion constraint with
pg_upgrade will get the same type of error.

Patch attached.  Should it be backpatched to 9.0 and 9.1?

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +
diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
new file mode 100644
index 644637c..6dc3d40
*** a/src/bin/pg_dump/pg_dump.c
--- b/src/bin/pg_dump/pg_dump.c
*************** dumpConstraint(Archive *fout, Constraint
*** 12926,12932 ****
              exit_nicely();
          }

!         if (binary_upgrade && !coninfo->condef)
              binary_upgrade_set_pg_class_oids(q, indxinfo->dobj.catId.oid, true);

          appendPQExpBuffer(q, "ALTER TABLE ONLY %s\n",
--- 12926,12932 ----
              exit_nicely();
          }

!         if (binary_upgrade)
              binary_upgrade_set_pg_class_oids(q, indxinfo->dobj.catId.oid, true);

          appendPQExpBuffer(q, "ALTER TABLE ONLY %s\n",

Re: pg_upgrade relation OID mismatches

From
Heikki Linnakangas
Date:
On 24.11.2011 07:01, Bruce Momjian wrote:
> Bruce Momjian wrote:
>> OK, that is a heap table.  My only guess is that the heap is being
>> created without binary_upgrade_next_heap_pg_class_oid being set.
>> Looking at the code, I can't see how the heap could be created without
>> this happening.  Another idea is that pg_dumpall isn't output the proper
>> value, but again, how is this data type different from the others.
>
> I have reproduced the failure and found it was code I added to pg_dump
> back in 9.0.  The code didn't set the index oid for exclusion constraint
> indexes.  Once these were added to the regression tests for range types
> recently, pg_upgrade threw an error.
>
> My assumption is that anyone trying to use an exclusion constraint with
> pg_upgrade will get the same type of error.
>
> Patch attached.  Should it be backpatched to 9.0 and 9.1?

If I understood correctly, pg_upgrade of a database with exclusion 
constraints won't work without this patch? In that case, it should be 
backpatched.

--   Heikki Linnakangas  EnterpriseDB   http://www.enterprisedb.com


Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Heikki Linnakangas wrote:
> On 24.11.2011 07:01, Bruce Momjian wrote:
> > Bruce Momjian wrote:
> >> OK, that is a heap table.  My only guess is that the heap is being
> >> created without binary_upgrade_next_heap_pg_class_oid being set.
> >> Looking at the code, I can't see how the heap could be created without
> >> this happening.  Another idea is that pg_dumpall isn't output the proper
> >> value, but again, how is this data type different from the others.
> >
> > I have reproduced the failure and found it was code I added to pg_dump
> > back in 9.0.  The code didn't set the index oid for exclusion constraint
> > indexes.  Once these were added to the regression tests for range types
> > recently, pg_upgrade threw an error.
> >
> > My assumption is that anyone trying to use an exclusion constraint with
> > pg_upgrade will get the same type of error.
> >
> > Patch attached.  Should it be backpatched to 9.0 and 9.1?
> 
> If I understood correctly, pg_upgrade of a database with exclusion 
> constraints won't work without this patch? In that case, it should be 
> backpatched.

Yes, that is my guess.  I will test it today or tomorrow.  I am
surprised we had _no_ exclusion constraint tests in the regression tests
until now.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Bruce Momjian wrote:
> Heikki Linnakangas wrote:
> > On 24.11.2011 07:01, Bruce Momjian wrote:
> > > Bruce Momjian wrote:
> > >> OK, that is a heap table.  My only guess is that the heap is being
> > >> created without binary_upgrade_next_heap_pg_class_oid being set.
> > >> Looking at the code, I can't see how the heap could be created without
> > >> this happening.  Another idea is that pg_dumpall isn't output the proper
> > >> value, but again, how is this data type different from the others.
> > >
> > > I have reproduced the failure and found it was code I added to pg_dump
> > > back in 9.0.  The code didn't set the index oid for exclusion constraint
> > > indexes.  Once these were added to the regression tests for range types
> > > recently, pg_upgrade threw an error.
> > >
> > > My assumption is that anyone trying to use an exclusion constraint with
> > > pg_upgrade will get the same type of error.
> > >
> > > Patch attached.  Should it be backpatched to 9.0 and 9.1?
> > 
> > If I understood correctly, pg_upgrade of a database with exclusion 
> > constraints won't work without this patch? In that case, it should be 
> > backpatched.
> 
> Yes, that is my guess.  I will test it today or tomorrow.  I am
> surprised we had _no_ exclusion constraint tests in the regression tests
> until now.

I do see EXCLUDE constraints in 9.0, so I need to do some more research:
CREATE TABLE circles (  c1 CIRCLE,  c2 TEXT,  EXCLUDE USING gist    (c1 WITH &&, (c2::circle) WITH &&)    WHERE
(circle_center(c1)<> '(0,0)'));
 

It seems it is only the range-type EXCLUDE constraints that are causing
a problem.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: pg_upgrade relation OID mismatches

From
Bruce Momjian
Date:
Bruce Momjian wrote:
> Bruce Momjian wrote:
> > Heikki Linnakangas wrote:
> > > On 24.11.2011 07:01, Bruce Momjian wrote:
> > > > Bruce Momjian wrote:
> > > >> OK, that is a heap table.  My only guess is that the heap is being
> > > >> created without binary_upgrade_next_heap_pg_class_oid being set.
> > > >> Looking at the code, I can't see how the heap could be created without
> > > >> this happening.  Another idea is that pg_dumpall isn't output the proper
> > > >> value, but again, how is this data type different from the others.
> > > >
> > > > I have reproduced the failure and found it was code I added to pg_dump
> > > > back in 9.0.  The code didn't set the index oid for exclusion constraint
> > > > indexes.  Once these were added to the regression tests for range types
> > > > recently, pg_upgrade threw an error.
> > > >
> > > > My assumption is that anyone trying to use an exclusion constraint with
> > > > pg_upgrade will get the same type of error.
> > > >
> > > > Patch attached.  Should it be backpatched to 9.0 and 9.1?
> > > 
> > > If I understood correctly, pg_upgrade of a database with exclusion 
> > > constraints won't work without this patch? In that case, it should be 
> > > backpatched.
> > 
> > Yes, that is my guess.  I will test it today or tomorrow.  I am
> > surprised we had _no_ exclusion constraint tests in the regression tests
> > until now.
> 
> I do see EXCLUDE constraints in 9.0, so I need to do some more research:
> 
>     CREATE TABLE circles (
>       c1 CIRCLE,
>       c2 TEXT,
>       EXCLUDE USING gist
>         (c1 WITH &&, (c2::circle) WITH &&)
>         WHERE (circle_center(c1) <> '(0,0)')
>     );
> 
> It seems it is only the range-type EXCLUDE constraints that are causing
> a problem.

OK, it turns out that exclusion contraints used in pre-9.2 regression
tests were deleted before the regression tests finished, which means
they were not tested by me.  (This might be a good reason _not_ to have
the regression tests removing objects before the scripts exit.)

I have applied the posted patch to 9.0, 9.1, and 9.2.  The good news is
that no one reported this failure in the field, and it would have
generated an error, rather than silently failing.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: pg_upgrade relation OID mismatches

From
Alvaro Herrera
Date:
Excerpts from Bruce Momjian's message of vie nov 25 17:05:09 -0300 2011:

> OK, it turns out that exclusion contraints used in pre-9.2 regression
> tests were deleted before the regression tests finished, which means
> they were not tested by me.  (This might be a good reason _not_ to have
> the regression tests removing objects before the scripts exit.)

Yeah, I vote for most/all tests keeping their objects instead of
dropping them, where sensible.  Also, some tests use a single table name
and create it and drop it just to create it again later with different
properties (I think the alter_table test is like this).  We should
avoid that.

--
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support