Hi,
Alvaro, added you because you were the original author for a lot of that
code. Fujii, you touched it last...
6) Should any part of the "reuse_stats" logic in table_recheck_autovac() be
kept?
With the shared memory stats patch, autovacuum can cheaply access individual
stats, so the whole scheme for avoiding stats accesses is moot.
So far the patchset had touched autovacuum.c a bit too lightly, removing the
autovac_refresh_stats() call and rephrasing a few comments, but not removing
e.g. the reuse_stats variable / branches in table_recheck_autovac. Which
doesn't seem great. I've just tried to go through and update the autovacuum.c
code and comments in light of the shared memory stats patch..
I don't really see a point in keeping any of it - but I was curious whether
anybody else does?
I'm still polishing, so I didn't want to send a whole new version with these
adjustments to the list yet, but you can see the state as of the time of
sending this email at [1].
Greetings,
Andres Freund
[1]
https://github.com/anarazel/postgres/commit/276c053110cfe71bf134519e8e4ab053e6d2a7f0#diff-3035fb5dace7bcd77f0eeafe32458cd808c5adb83d62ebdf54f0170cf7db93e7