On Mon, Feb 22, 2021 at 11:04 PM Markus Wanner <mar...@bluegap.ch> wrote: > > On 04.01.21 09:18, Amit Kapila wrote: > > Thanks, I have pushed the 0001* patch after making the above and a few > > other cosmetic modifications. > > That commit added the following snippet to the top of > ReorderBufferFinishPrepared: > > txn = ReorderBufferTXNByXid(rb, xid, true, NULL, commit_lsn, false); > > /* unknown transaction, nothing to do */ > if (txn == NULL) > return; > > Passing true for the create argument seems like an oversight. I think > this should pass false and not ever (have to) create a ReorderBufferTXN > entry. >
Right, I'll push a fix for this. Thanks for the report! -- With Regards, Amit Kapila.
