diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-02-12 05:39:55 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-02-12 05:39:55 +0000 |
| commit | 7bed6df184e934bf3e3217ecbce9240051a733e3 (patch) | |
| tree | 1b3dcf4c4bd88c45c884ea994d351eecbb3355f6 /src/backend/commands/vacuum.c | |
| parent | 1ecd035b318f949090be59d1e1826b3d0c8f03e9 (diff) | |
| download | postgresql-7bed6df184e934bf3e3217ecbce9240051a733e3.tar.gz postgresql-7bed6df184e934bf3e3217ecbce9240051a733e3.zip | |
Check only live tuples for OID validity, to avoid issues after ALTER
TABLE. Per recent discussion.
Diffstat (limited to 'src/backend/commands/vacuum.c')
| -rw-r--r-- | src/backend/commands/vacuum.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/backend/commands/vacuum.c b/src/backend/commands/vacuum.c index a0dd09df215..760c10bcc80 100644 --- a/src/backend/commands/vacuum.c +++ b/src/backend/commands/vacuum.c @@ -13,7 +13,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/vacuum.c,v 1.273 2004/02/10 03:42:43 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/vacuum.c,v 1.274 2004/02/12 05:39:55 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1178,6 +1178,14 @@ scan_heap(VRelStats *vacrelstats, Relation onerel, Assert(tuple.t_data->t_infomask & HEAP_XMIN_COMMITTED); pgchanged = true; } + + /* + * Other checks... + */ + if (onerel->rd_rel->relhasoids && + !OidIsValid(HeapTupleGetOid(&tuple))) + elog(WARNING, "relation \"%s\" TID %u/%u: OID is invalid", + relname, blkno, offnum); break; case HEAPTUPLE_RECENTLY_DEAD: @@ -1244,14 +1252,6 @@ scan_heap(VRelStats *vacrelstats, Relation onerel, if (sv_infomask != tuple.t_data->t_infomask) pgchanged = true; - /* - * Other checks... - */ - if (onerel->rd_rel->relhasoids && - !OidIsValid(HeapTupleGetOid(&tuple))) - elog(WARNING, "relation \"%s\" TID %u/%u: OID is invalid", - relname, blkno, offnum); - if (tupgone) { ItemId lpp; |