On 27/11/2018 10:10, Sergei Kornilov wrote:
> Hello
>
>>> - recovery_target = immediate was replaced with recovery_target_immediate bool GUC
>>
>> Why?
> Due this comment: https://www.postgresql.org/message-id/20181126172118.GY3415%40tamriel.snowman.net
>> I've not been following this very closely, but seems like
>> recovery_target_string is a bad idea.. Why not just make that
>> 'recovery_target_immediate' and make it a boolean? Having a GUC that's
>> only got one valid value seems really odd.
It is a bit odd, but that's the way it's been, and I don't see a reason
to change it as part of this fix. We are attempting to fix the way the
GUC parameters are parsed, not change the name and meaning of the
parameters themselves.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
