• Kevin Grittner's avatar
    Ensure ANALYZE phase is not skipped because of canceled truncate. · 5fc89376
    Kevin Grittner authored
    Patch b19e4250 attempted to
    preserve existing behavior regarding statistics generation in the
    case that a truncation attempt was canceled due to lock conflicts.
    It failed to do this accurately in two regards: (1) autovacuum had
    previously generated statistics if the truncate attempt failed to
    initially get the lock rather than having started the attempt, and
    (2) the VACUUM ANALYZE command had always generated statistics.
    
    Both of these changes were unintended, and are reverted by this
    patch.  On review, there seems to be consensus that the previous
    failure to generate statistics when the truncate was terminated
    was more an unfortunate consequence of how that effort was
    previously terminated than a feature we want to keep; so this
    patch generates statistics even when an autovacuum truncation
    attempt terminates early.  Another unintended change which is kept
    on the basis that it is an improvement is that when a VACUUM
    command is truncating, it will the new heuristic for avoiding
    blocking other processes, rather than keeping an
    AccessExclusiveLock on the table for however long the truncation
    takes.
    
    Per multiple reports, with some renaming per patch by Jeff Janes.
    
    Backpatch to 9.0, where problem was created.
    5fc89376
vacuumlazy.c 51.4 KB