On Thu, 2009-02-12 at 09:50 +0200, Heikki Linnakangas wrote:
> It occurs to me that we don't need this patch for hot standby if we
> abuse the main xid array (SnapshotData.xip) to store the unobserved xids
> instead of the subxid array. That one is always scanned in
> XidInMVCCSnapshot. I think we should do that rather than try to salvage
> this patch.
At this stage, yes.
> So far so good, but what about all the other callers of
> SubTransGetParent()? For example, XactLockTableWait will fail an
> assertion if asked to wait on a subtransaction which is then released.
I agree that it could fail the assertion, though it is clear that the
assertion should now be removed.
The logic is: if there is no lock table entry for that xid *and* it is
not in progress *and* it is not in pg_subtrans, then it must have been
an aborted subtransaction of a currently active xact or it has otherwise
completed.
I think we can rework the other aspects also.
-- Simon Riggs www.2ndQuadrant.comPostgreSQL Training, Services and Support