summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>2013-11-18 09:51:09 +0200
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>2013-11-18 10:12:22 +0200
commit3379263b6ded774acb96288d1e67caa37dcba0de (patch)
tree41c53b39e6454d68f3c470f82d59f69d0f11c181
parentb18882aed6bf7190677c351199f95d409559bbc0 (diff)
Count locked pages that don't need vacuuming as scanned.
Previously, if VACUUM skipped vacuuming a page because it's pinned, it didn't count that page as scanned. However, that meant that relfrozenxid was not bumped up either, which prevented anti-wraparound vacuum from doing its job. Report by Миша Тюрин, analysis and patch by Sergey Burladyn and Jeff Janes. Backpatch to 9.2, where the skip-locked-pages behavior was introduced.
-rw-r--r--src/backend/commands/vacuumlazy.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/backend/commands/vacuumlazy.c b/src/backend/commands/vacuumlazy.c
index 28f169e59c5..782f59e64d2 100644
--- a/src/backend/commands/vacuumlazy.c
+++ b/src/backend/commands/vacuumlazy.c
@@ -589,6 +589,7 @@ lazy_scan_heap(Relation onerel, LVRelStats *vacrelstats,
if (!lazy_check_needs_freeze(buf))
{
UnlockReleaseBuffer(buf);
+ vacrelstats->scanned_pages++;
continue;
}
LockBuffer(buf, BUFFER_LOCK_UNLOCK);