On Tue, Nov 28, 2017 at 1:55 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> It might be okay to put such a short-circuit in at a lower level,
> eg within the btree AM. I don't remember at the moment whether
> a btree vacuum scan accomplishes anything much if there are no dead
> tuples.
>
> One thing that I think it does do is update the index's pg_class stats
> (relpages/reltuples). Maybe it's okay to skip that in this particular
> scenario, trusting that auto-analyze would fix it, but that seems
> worthy of debate.
One of the sticking points with Sawada-san's patch, that I tried to
work through with him, is that we store XIDs in deleted B-Tree pages,
for the RecentGlobalXmin recycling interlock thing. The assumption is
that anti-wraparound VACUUM doesn't have to worry about those Xids as
a special case, because VACUUM will get around to actually reclaiming
them for the FSM the next time around (maybe that's the VACUUM that
enables advancing the epoch).
We cannot break that assumption, and no easy fixes suggest themselves.
--
Peter Geoghegan
