Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl> writes:
> I just figured that if we let LockRelation use GetCurrentTransactionId()
> then the wrong thing happens if we let large objects survive
> subtransaction commit/abort.
> So I have changed it to use GetTopTransactionId() instead. Is that OK
> with everybody?
No, at least not if you made that a global change. Doing it that way
will mean that a failed subtransaction will not release its locks, no?
regards, tom lane