Alvaro Herrera <alvherre@dcc.uchile.cl> writes:
> With this comment, I take it you'd disagree with my recoding of
> TransactionIdIsCurrentTransactionId().
> The current code has to scan only the xid's in each PGPROC struct.
> However I had to rewrite it to peek at pg_subtrans, and this is done
> while the SInvalLock is share-locked. pg_subtrans may need to do some
> I/O to get the data, and there could be multiple queries, depending on
> the nesting level.
You're right, I think that's really awful :-(
regards, tom lane