On Mon, Sep 28, 2026 at 2:34 PM Sami Imseih <[email protected]> wrote:
>
> Hi Bharath,
>
> Thanks for the feedback!
>
> > 1/ At the end of parallel_vacuum_main(), do we need to gate the
> > reporting of any remaining delay time on the accumulated delay time
> > variable rather than on the GUC?
>
> track_cost_delay_timing gates delay timing reporting elsewhere, so we should 
> not
> deviate from that. If the GUC is off by then, we should not accumulate
> any timing
> anyhow, even if parallel_vacuum_worker_delay_ns > 0

IIUC the remaining parallel_vacuum_worker_delay_ns was accumulated
when the track_cost_delay_timing was enabled. Shouldn't we report it
as well?

Regards,

-- 
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com


Reply via email to