On Tue, Oct 19, 2021 at 9:06 AM Nitin Jadhav <nitinjadhavpostg...@gmail.com> wrote: > Thanks for sharing the patch. Overall approach looks good to me. But > just one concern about using enable_timeout_every() functionality. As > per my understanding the caller should calculate the first scheduled > timeout (now + interval) and pass it as the second argument but this > is not the same in 'v2-0002-Quick-testing-hack.patch'. Anyways I have > done the changes as I have mentioned (like now + interval). Kindly > correct me if I am wrong. I am attaching 2 patches here. > 'v19-0001-Add-enable_timeout_every-to-fire-the-same-timeout.patch' is > the same as Robert's v2 patch. I have rebased my patch on top of this > and it is 'v19-0002-startup-progress.patch'.
This version looks fine, so I have committed it (and my enable_timeout_every patch also, as a necessary prerequisite). Thanks, -- Robert Haas EDB: http://www.enterprisedb.com
