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
