• Fujii Masao's avatar
    Fix mesurement of elapsed time during truncating heap in VACUUM. · 00749197
    Fujii Masao authored
    VACUUM may truncate heap in several batches. The activity report
    is logged for each batch, and contains the number of pages in the table
    before and after the truncation, and also the elapsed time during
    the truncation. Previously the elapsed time reported in each batch was
    the total elapsed time since starting the truncation until finishing
    each batch. For example, if the truncation was processed dividing into
    three batches, the second batch reported the accumulated time elapsed
    during both first and second batches. This is strange and confusing
    because the number of pages in the table reported together is not
    total. Instead, each batch should report the time elapsed during
    only that batch.
    
    The cause of this issue was that the resource usage snapshot was
    initialized only at the beginning of the truncation and was never
    reset later. This commit fixes the issue by changing VACUUM so that
    the resource usage snapshot is reset at each batch.
    
    Back-patch to all supported branches.
    
    Reported-by: Tatsuhito Kasahara
    Author: Tatsuhito Kasahara
    Reviewed-by: Masahiko Sawada, Fujii Masao
    Discussion: https://postgr.es/m/CAP0=ZVJsf=NvQuy+QXQZ7B=ZVLoDV_JzsVC1FRsF1G18i3zMGg@mail.gmail.com
    00749197
vacuumlazy.c 105 KB