On Fri, Aug 27, 2021 at 10:05 AM Andrey Borodin <x4mmm@yandex-team.ru> wrote:
> There is a small bug
> + /*
> + * Take restore_command from the postgresql.conf only if it is not already
> + * provided as a command line option.
> + */
> + if (!restore_wal && restore_command == NULL)
> return;
>
> I think we should use condition (!restore_wal || restore_command != NULL).
>
Yes, you are right, thanks. Attached is a fixed version. Tests were
passing since PostgresNode->enable_restoring is adding restore_command
to the postgresql.conf anyway.
>
> Besides this patch looks good and is ready for committer IMV.
>
--
Alexey Kondratov