diff options
| author | Simon Riggs <simon@2ndQuadrant.com> | 2010-04-21 19:53:24 +0000 |
|---|---|---|
| committer | Simon Riggs <simon@2ndQuadrant.com> | 2010-04-21 19:53:24 +0000 |
| commit | f6e092701c6ac7bb98ed88a769daa13d3a1755d9 (patch) | |
| tree | 5dfeb980d84f0a92160986edd145eee422ffe628 | |
| parent | 0192abc4d7792117d53c6c894eeeaa83b02802c5 (diff) | |
| download | postgresql-f6e092701c6ac7bb98ed88a769daa13d3a1755d9.tar.gz postgresql-f6e092701c6ac7bb98ed88a769daa13d3a1755d9.zip | |
Only send cleanup_info messages if VACUUM removes any tuples.
There is no other purpose for this message type than to report
the latestRemovedXid of removed tuples, prior to index scans.
Removes overlooked path for sending invalid latestRemovedXid.
Fixes buildfarm failure on centaur.
| -rw-r--r-- | src/backend/commands/vacuumlazy.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/backend/commands/vacuumlazy.c b/src/backend/commands/vacuumlazy.c index 6e807e3b479..0b4047657ea 100644 --- a/src/backend/commands/vacuumlazy.c +++ b/src/backend/commands/vacuumlazy.c @@ -29,7 +29,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/vacuumlazy.c,v 1.133 2010/04/21 17:20:56 sriggs Exp $ + * $PostgreSQL: pgsql/src/backend/commands/vacuumlazy.c,v 1.134 2010/04/21 19:53:24 sriggs Exp $ * *------------------------------------------------------------------------- */ @@ -274,9 +274,12 @@ vacuum_log_cleanup_info(Relation rel, LVRelStats *vacrelstats) if (rel->rd_istemp || !XLogIsNeeded()) return; - Assert(TransactionIdIsValid(vacrelstats->latestRemovedXid)); + if (vacrelstats->tuples_deleted > 0) + { + Assert(TransactionIdIsValid(vacrelstats->latestRemovedXid)); - (void) log_heap_cleanup_info(rel->rd_node, vacrelstats->latestRemovedXid); + (void) log_heap_cleanup_info(rel->rd_node, vacrelstats->latestRemovedXid); + } } /* |