From 9975c683b102d06ed5d5ab799eaba0d00a9ff38c Mon Sep 17 00:00:00 2001 From: Simon Riggs Date: Thu, 9 Dec 2010 10:20:49 +0000 Subject: [PATCH] Self review of previous patch. Fix assumption that xmax >= xmin. --- src/backend/access/heap/heapam.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index cfbbc6301e..4fe3a73910 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -3809,11 +3809,9 @@ HeapTupleHeaderAdvanceLatestRemovedXid(HeapTupleHeader tuple, !(tuple->t_infomask & HEAP_XMIN_INVALID) && TransactionIdDidCommit(xmin))) { - if (TransactionIdFollows(xmax, xmin)) - { - if (TransactionIdFollows(xmax, *latestRemovedXid)) + if (xmax != xmin && + TransactionIdFollows(xmax, *latestRemovedXid)) *latestRemovedXid = xmax; - } } /* *latestRemovedXid may still be invalid at end */ -- 2.40.0