diff options
| author | Heikki Linnakangas | 2008-09-01 18:53:03 +0000 |
|---|---|---|
| committer | Heikki Linnakangas | 2008-09-01 18:53:03 +0000 |
| commit | 8a2619fcd973e213a7cfbb72b8b802d00a54eb5a (patch) | |
| tree | f683171422e912b4d9be58c72e37ac44b488378c | |
| parent | d01243fe9746f47fc47342b61a7580efe806cc15 (diff) | |
HeapTupleHeaderAdjustCmax made the incorrect assumption that the raw
command id is the cmin, when it can in fact be a combo cid. That made rows
incorrectly invisible to a transaction where a tuple was deleted by multiple
aborted subtransactions.
Report and patch Karl Schnaitter. Back-patch to 8.3, where combo cids was
introduced.
| -rw-r--r-- | src/backend/utils/time/combocid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/time/combocid.c b/src/backend/utils/time/combocid.c index cc130806ed..cadd7233ca 100644 --- a/src/backend/utils/time/combocid.c +++ b/src/backend/utils/time/combocid.c @@ -155,7 +155,7 @@ HeapTupleHeaderAdjustCmax(HeapTupleHeader tup, if (!(tup->t_infomask & HEAP_XMIN_COMMITTED) && TransactionIdIsCurrentTransactionId(HeapTupleHeaderGetXmin(tup))) { - CommandId cmin = HeapTupleHeaderGetRawCommandId(tup); + CommandId cmin = HeapTupleHeaderGetCmin(tup); *cmax = GetComboCommandId(cmin, *cmax); *iscombo = true; |