Re: Supply restore_command to pg_rewind via CLI argument

On Fri, Aug 27, 2021 at 10:05 AM Andrey Borodin <x4mmm(at)yandex-team(dot)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