Dear Alyona,
Thanks for updating the patch!
Sadly, your patch cannot be applied cleanly. Even after the manual merge, it
could not
be built. Maybe `dbinfo` should be `dbinfos.dbinfo`. Obtained message is
written in [1].
(cfbot seemed not to run correctly)
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?
```
recovery_target_inclusive true
recovery_target_action pause
recovery_target_lsn ""
```
[1]
```
../postgres/src/bin/pg_basebackup/pg_createsubscriber.c: In function ‘main’:
../postgres/src/bin/pg_basebackup/pg_createsubscriber.c:2526:31: error:
‘dbinfo’ undeclared (first use in this function); did you mean ‘dbinfos’?
2526 | reset_recovery_params(dbinfo, subscriber_dir);
| ^~~~~~
| dbinfos
```
Best regards,
Hayato Kuroda
FUJITSU LIMITED
