> The below patch runs single mode Postgres if needed to make sure the target > is cleanly shutdown. A new option is added (off by default). > v2-0001-Ensure-target-clean-shutdown-at-beginning-of-pg_r.patch
Why do we need an option for this? Is there a reason not to do this unconditionally?
There is concern about this (see previous emails in this thread). On greenplum (MPP DB based on Postgres),
we unconditionally do this. I'm not sure about usually how Postgres users do this when there is an unclean shutdown,
but providing an option seem to be safer to avoid breaking existing script/service whatever. If many people
think this option is unnecessary, I'm fine to remove the option and keep the code logic.