Andres Freund <andres@2ndquadrant.com> writes:
> Afaics before the commit a LockGXact() did a GetTopTransactionId(),
> afterwards it doesn't anymore. That'd fit, right?
Ah, I think you've got it. Now I'm worried. I have a very vague
recollection that that behavior (forcing our own COMMIT to be written
after a COMMIT PREPARED) was intentional. I don't recall exactly
why though.
xact.h points out
/*
* COMMIT_PREPARED and ABORT_PREPARED are identical to COMMIT/ABORT records
* except that we have to store the XID of the prepared transaction explicitly
* --- the XID in the record header will be for the transaction doing the
* COMMIT PREPARED or ABORT PREPARED command.
*/
and what this change means is that there isn't actually any valid XID in
the record header, which at least means that comment needs an update.
But I'm concerned about the knock-on aspects of that. In particular
I wonder if we were expecting commit of the surrounding transaction to
result in a WAL flush or anything like that. The proposal you made
recently to not sync non-XID-assigning WAL records would affect this.
As far as the timestamp aspects go, I wonder if we were thinking that
commit/abort prepared might contain stale timestamps by design. They
don't --- in the current coding, the timestamp is the time of creating the
WAL record, not of the prepare --- but it's not that hard to imagine that
time-of-prepare might have been the original intent.
regards, tom lane