diff options
| author | Simon Riggs <simon@2ndQuadrant.com> | 2010-12-09 10:20:49 +0000 |
|---|---|---|
| committer | Simon Riggs <simon@2ndQuadrant.com> | 2010-12-09 10:20:49 +0000 |
| commit | 9975c683b102d06ed5d5ab799eaba0d00a9ff38c (patch) | |
| tree | 9564bdafac0f91a326fa956230bd60d0e6c61418 | |
| parent | b9075a6d2f9b07a00262a670dd60272904c79dce (diff) | |
| download | postgresql-9975c683b102d06ed5d5ab799eaba0d00a9ff38c.tar.gz postgresql-9975c683b102d06ed5d5ab799eaba0d00a9ff38c.zip | |
Self review of previous patch. Fix assumption that xmax >= xmin.
| -rw-r--r-- | src/backend/access/heap/heapam.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index cfbbc6301e3..4fe3a739109 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 */ |