On 2026-Sep-24, Rui Zhao wrote: > Thanks. Going back to my original loop with the limit changed to > running->xcnt makes sense.
OK, I have pushed this to all branches, including the test cases on branches where they work. Thanks all for the work on this issue! I'm going to mark the two pg19 open items (!!) as done. > On 2026-Sep-21 at 15:35 UTC, Alvaro Herrera wrote: > > I think we should just go up to running->xcnt > > only; > > Yes, scanning the subxids was unnecessary. My previous explanation > addressed overflow, but waiting for the parent covers the children in > the non-overflow case too. A note on the top-level-first ordering in > RunningTransactionsData would make that dependency explicit. I didn't add this ... let's consider that as follow-on work, but we don't need it to be backpatched. > > maybe we should add > > something in SnapBuildBuildSnapshot() > > Agreed. The explanation of why historic snapshots need no wait for > transactions to finish belongs there. The comment in > SnapBuildInitialSnapshot() can then focus on why the conversion to a > normal MVCC snapshot needs the wait. Done that way -- I hope the explanations are clear. -- Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
