> On 27 Nov 2023, at 14:06, Masahiko Sawada <sawada.m...@gmail.com> wrote:
> Is it true that we can modify the timeout after creating > BackgroundPsql object? If so, it seems we don't need to introduce the > new timeout argument. But how? I can't remember if that's leftovers that incorrectly remains from an earlier version of the BackgroundPsql work, or if it's a very bad explanation of ->set_query_timer_restart(). The timeout will use the timeout_default value and that cannot be overridden, it can only be reset per query. With your patch the timeout still cannot be changed, but at least set during start which seems good enough until there are tests warranting more complexity. The docs should be corrected to reflect this in your patch. -- Daniel Gustafsson
