diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2013-01-24 16:04:48 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2013-01-24 16:10:02 -0300 |
| commit | 74ebba84aeb6b0e6320f9b992527715f7e2ace8e (patch) | |
| tree | 01c391b0c86e01bb7c2319b6fa9896b72f4d711f /src | |
| parent | 34da700405737be055d1b3d310af23a98da72e8e (diff) | |
| download | postgresql-74ebba84aeb6b0e6320f9b992527715f7e2ace8e.tar.gz postgresql-74ebba84aeb6b0e6320f9b992527715f7e2ace8e.zip | |
Redefine HEAP_XMAX_IS_LOCKED_ONLY
Tuples marked SELECT FOR UPDATE in a cluster that's later processed by
pg_upgrade would have a different infomask bit pattern than those
produced by 9.3dev; that bit pattern was being seen as "dead" by HEAD
(because they would fail the "is this tuple locked" test, and so the
visibility rules would thing they're updated, even though there's no
HEAP_UPDATED version of them). In other words, some rows could silently
disappear after pg_upgrade.
With this new definition, those tuples become visible again.
This is breakage resulting from my commit 0ac5ad5134.
Diffstat (limited to 'src')
| -rw-r--r-- | src/include/access/htup_details.h | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/include/access/htup_details.h b/src/include/access/htup_details.h index 6a28d8ed74e..cd01ecdba7d 100644 --- a/src/include/access/htup_details.h +++ b/src/include/access/htup_details.h @@ -189,14 +189,19 @@ struct HeapTupleHeaderData #define HEAP_XACT_MASK 0xFFF0 /* visibility-related bits */ /* - * A tuple is only locked (i.e. not updated by its Xmax) if it the - * HEAP_XMAX_LOCK_ONLY bit is set. + * A tuple is only locked (i.e. not updated by its Xmax) if the + * HEAP_XMAX_LOCK_ONLY bit is set; or, for pg_upgrade's sake, if the Xmax is + * not a multi and the EXCL_LOCK bit is set. * * See also HeapTupleHeaderIsOnlyLocked, which also checks for a possible * aborted updater transaction. + * + * Beware of multiple evaluations of the argument. */ #define HEAP_XMAX_IS_LOCKED_ONLY(infomask) \ - ((infomask) & HEAP_XMAX_LOCK_ONLY) + (((infomask) & HEAP_XMAX_LOCK_ONLY) || \ + (((infomask) & (HEAP_XMAX_IS_MULTI | HEAP_LOCK_MASK)) == HEAP_XMAX_EXCL_LOCK)) + /* * Use these to test whether a particular lock is applied to a tuple */ |