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/vacuumlazy.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/vacuumlazy.c')
| -rw-r--r-- | src/backend/commands/vacuumlazy.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/backend/commands/vacuumlazy.c b/src/backend/commands/vacuumlazy.c index 2d85b652937..1ce33e10458 100644 --- a/src/backend/commands/vacuumlazy.c +++ b/src/backend/commands/vacuumlazy.c @@ -31,7 +31,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/vacuumlazy.c,v 1.37 2004/02/10 03:42:44 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/vacuumlazy.c,v 1.38 2004/02/12 05:39:55 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -336,6 +336,14 @@ lazy_scan_heap(Relation onerel, LVRelStats *vacrelstats, 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: @@ -360,14 +368,6 @@ lazy_scan_heap(Relation onerel, LVRelStats *vacrelstats, 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) { lazy_record_dead_tuple(vacrelstats, &(tuple.t_self)); |