]> granicus.if.org Git - postgresql/commit
Don't try to set InvalidXid as page pruning hint
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 27 Nov 2013 20:47:16 +0000 (17:47 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 28 Nov 2013 15:00:12 +0000 (12:00 -0300)
commitc235a6a589bbd0a24e54fdb0df28979c9fb09463
treec56def9344153018922a2b1122ed887012e78520
parente518fa7adfc28e0d0a99fe4e5711386d9a5c6532
Don't try to set InvalidXid as page pruning hint

If a transaction updates/deletes a tuple just before aborting, and a
concurrent transaction tries to prune the page concurrently, the pruner
may see HeapTupleSatisfiesVacuum return HEAPTUPLE_DELETE_IN_PROGRESS,
but a later call to HeapTupleGetUpdateXid() return InvalidXid.  This
would cause an assertion failure in development builds, but would be
otherwise Mostly Harmless.

Fix by checking whether the updater Xid is valid before trying to apply
it as page prune point.

Reported by Andres in 20131124000203.GA4403@alap2.anarazel.de
src/backend/access/heap/pruneheap.c