On Mon, Mar 07, 2022 at 10:17:41PM +0800, Julien Rouhaud wrote: > On Mon, Mar 07, 2022 at 01:27:40PM +0000, Simon Riggs wrote: >> The patch was posted because TransactionLogFetch() has a cache, yet >> SubTransGetTopmostTransaction() does not, yet the argument should be >> identical in both cases. > > I totally agree with that.
Agreed as well. That's worth doing in isolation and that will save some lookups of pg_subtrans anyway while being simple. As mentioned upthread, this needed an indentation, and the renaming of cachedFetchXid to cachedFetchSubXid looks adapted. So.. Applied all those things. -- Michael
signature.asc
Description: PGP signature
