Thread

Re: Resetting recovery target parameters in pg_createsubscriber

Alena Vinter <dlaaren8@gmail.com> — 2025-09-05T05:51:12Z

Dear Hayato,

Thank you for the review! My apologies for the error in the patch -- it
looks like I accidentally modified it before sending =(. I've attached the
fixed versions below.

> Regarding patch content, your patch restores the postgresql.auto.conf
after the
> command runs. Initially I felt that it is enough to set below GUCs
becasue only
> they are changed from the default. Is there a reason why you fully
restore them?

I just found it easier to restore the original state of
'postgresql.auto.conf', as removing parameters from the file resets them to
their default values. This approach achieves the same final state without
having to explicitly set each one.