Re: [WIP] The relminxid addition, try 3

Alvaro Herrera wrote:

Hi,

> Here is a reworked relminxid patch.  I added XLOG operations for heap
> unfreeze and database unfreeze.  The latter happens when someone
> connects to a frozen database.  The former happens when a LockRelation()
> is called on a frozen relation, and the lock is stronger than
> AccessShare.

I've been further reworking this patch.  It is basically as discussed:
tables are unfrozen when LockRelation() is called, databases are
unfrozen when they are connected to.  Those operations are performed
using heap_inplace_update, so we no longer need to take particular steps
to XLog them.

There are two new columns in pg_class (relminxid and relvacuumxid).
Both of them can be set to FrozenTransactionId.  The minimum of each is
used to determine database-wide datminxid and datvacuumxid.  Those can
also be to FrozenXid.  (Note that it seems to be problematic that all
databases have FrozenXids; however, this can only happen in a standalone
backend, when you VACUUM FREEZE a database and all other databases are
already frozen.  So we can truncate to CurrentTransactionId without
problem.)

However there's a big problem once again but I don't know how to fix it.
I don't know why this didn't show up before.  Consider the following
scenario:

- pg_attribute is frozen
- pg_class is frozen

CREATE TABLE foo (a int);

for some unknown reason, an inval message involving relation foo seems
to be emitted.

heap_unfreeze(pg_class)
  CommandCounterIncrement()
heap_unfreeze(pg_attribute)
  CommandCounterIncrement()
... insert the pg_attribute rows ...

During one of those CommandCounterIncrement calls, the "foo" relation
receives an invalidation message and the system tries to rebuild the
RelationDesc.  But at this point the pg_attribute rows are not there
yet, so the rebuild fails because it can't find them:

$ runpg 10relminxid client
Bienvenido a psql 8.2devel, la terminal interactiva de PostgreSQL.

Digite:  \copyright para ver los términos de distribución
       \h para ayuda de comandos SQL
       \? para ayuda de comandos psql
       \g o or termine con punto y coma para ejecutar una consulta
       \q para salir

alvherre=# vacuum freeze ;
alvherre=# \q

$ runpg 10relminxid client
Bienvenido a psql 8.2devel, la terminal interactiva de PostgreSQL.

Digite:  \copyright para ver los términos de distribución
       \h para ayuda de comandos SQL
       \? para ayuda de comandos psql
       \g o or termine con punto y coma para ejecutar una consulta
       \q para salir

alvherre=# create table hmmmm (a int);
NOTICE:  unfreezing pg_type
NOTICE:  unfreezing pg_depend
NOTICE:  unfreezing pg_shdepend
NOTICE:  unfreezing pg_attribute
ERROR:  catalog is missing 1 attribute(s) for relid 16409


Strangely this doesn't happen if I create the table right away at the
same connection that did the VACUUM FREEZE.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.