Self review of previous patch. Fix assumption that xmax >= xmin.

9975c683 · Simon Riggs · b9075a6d · 9975c683
Commit 9975c683 authored Dec 09, 2010 by Simon Riggs
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

src/backend/access/heap/heapam.c src/backend/access/heap/heapam.c +2 -4

No files found.
--- 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 */