• A
    Initialize the minimum frozen Xid in vac_update_datfrozenxid using · d53a5668
    Alvaro Herrera 提交于
    GetOldestXmin() instead of RecentGlobalXmin; this is safer because we do not
    depend on the latter being correctly set elsewhere, and while it is more
    expensive, this code path is not performance-critical.  This is a real
    risk for autovacuum, because it can execute whole cycles without doing
    a single vacuum, which would mean that RecentGlobalXmin would stay at its
    initialization value, FirstNormalTransactionId, causing a bogus value to be
    inserted in pg_database.  This bug could explain some recent reports of
    failure to truncate pg_clog.
    
    At the same time, change the initialization of RecentGlobalXmin to
    InvalidTransactionId, and ensure that it's set to something else whenever
    it's going to be used.  Using it as FirstNormalTransactionId in HOT page
    pruning could incur in data loss.  InitPostgres takes care of setting it
    to a valid value, but the extra checks are there to prevent "special"
    backends from behaving in unusual ways.
    
    Per Tom Lane's detailed problem dissection in 29544.1221061979@sss.pgh.pa.us
    d53a5668
heapam.c 133.3 KB