Alvaro Herrera <alvherre@alvh.no-ip.org> 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
