diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-09-17 18:09:55 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-09-17 18:09:55 +0000 |
| commit | 054b78ba38e7bd11db1725ede38675b5cfaebe0b (patch) | |
| tree | b0ad5d656e795365556d7896080f523c0f06d246 | |
| parent | 12ce26f8da1f309218073f5d505284829f29a02b (diff) | |
| download | postgresql-054b78ba38e7bd11db1725ede38675b5cfaebe0b.tar.gz postgresql-054b78ba38e7bd11db1725ede38675b5cfaebe0b.zip | |
Now that xmax and cmin are distinct fields again, we should zero xmax when
creating a new tuple. This is just for debugging sanity, though, since
nothing should be paying any attention to xmax when the HEAP_XMAX_INVALID
bit is set.
| -rw-r--r-- | src/backend/access/heap/heapam.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index d59e9405695..b8d38cc86ba 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/heap/heapam.c,v 1.175 2004/09/16 16:58:25 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/access/heap/heapam.c,v 1.176 2004/09/17 18:09:55 tgl Exp $ * * * INTERFACE ROUTINES @@ -1142,7 +1142,8 @@ heap_insert(Relation relation, HeapTuple tup, CommandId cid) tup->t_data->t_infomask |= HEAP_XMAX_INVALID; HeapTupleHeaderSetXmin(tup->t_data, xid); HeapTupleHeaderSetCmin(tup->t_data, cid); - HeapTupleHeaderSetCmax(tup->t_data, 0); /* zero out Datum fields */ + HeapTupleHeaderSetXmax(tup->t_data, 0); /* zero out Datum fields */ + HeapTupleHeaderSetCmax(tup->t_data, 0); /* for cleanliness */ tup->t_tableOid = relation->rd_id; /* @@ -1608,7 +1609,8 @@ l2: newtup->t_data->t_infomask |= (HEAP_XMAX_INVALID | HEAP_UPDATED); HeapTupleHeaderSetXmin(newtup->t_data, xid); HeapTupleHeaderSetCmin(newtup->t_data, cid); - HeapTupleHeaderSetCmax(newtup->t_data, 0); /* zero out Datum fields */ + HeapTupleHeaderSetXmax(newtup->t_data, 0); /* zero out Datum fields */ + HeapTupleHeaderSetCmax(newtup->t_data, 0); /* for cleanliness */ /* * If the toaster needs to be activated, OR if the new tuple will not |