Alvaro Herrera <[email protected]> wrote: > On 2026-Apr-05, Antonin Houska wrote: > > > ok, maybe just skip the whole cleanup in that special case. > > Hmm, should we make this test only in the db_specific case? Doing it > unconditionally makes me a bit nervous (maybe because I don't fully > understand historic snapshot building).
I thought about adding Assert(db_specific) in front of the new return statement. So what you suggest makes sense to me. As far as I understand, the xl_running_xacts record is not directly involved in the snapshot build. Rather, the list of XIDs for snapshots is created and updated by processing COMMIT and ABORT records. -- Antonin Houska Web: https://www.cybertec-postgresql.com
