Re: Add shutdown_at_recovery_target option to recovery.conf

<div dir="ltr"><br /><div class="gmail_extra"><br /><div class="gmail_quote">On Sat, Dec 6, 2014 at 12:49 AM, Robert
Haas<span dir="ltr"><<a href="mailto:robertmhaas@gmail.com" target="_blank">robertmhaas@gmail.com</a>></span>
wrote:<br/><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span
class="">OnThu, Dec 4, 2014 at 8:45 AM, Michael Paquier<br /> <<a
href="mailto:michael.paquier@gmail.com">michael.paquier@gmail.com</a>>wrote:<br /> >> Here is patch which
renamesaction_at_recovery_target to<br /> >> recovery_target_action everywhere.<br /> > Thanks, Looks good to
me.<br/> ><br /> > A couple of things that would be good to document as well in<br /> >
recovery-config.sgml:<br/> > - the fact that pause_at_recovery_target is deprecated.<br /><br /></span>Why not just
removeit altogether?  I don't think the<br /> backward-compatibility break is enough to get excited about, or to<br />
justifythe annoyance of carrying an extra parameter that does (part<br /> of) the same thing.<br
/></blockquote></div>Atleast we won't forget removing in the future something that has been marked as deprecated for
years.So +1 here for a simple removal, and a mention in the future release notes.<br />-- <br /><div
class="gmail_signature">Michael<br/></div></div></div>