Re: pgsql: Integrate recovery.conf into postgresql.conf - Mailing list pgsql-hackers

From David Steele
Subject Re: pgsql: Integrate recovery.conf into postgresql.conf
Date
Msg-id fd69fa25-7ad3-dafa-b4a9-cac33bb9c70f@pgmasters.net
Whole thread Raw
In response to Re: pgsql: Integrate recovery.conf into postgresql.conf  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
On 11/26/18 1:15 PM, Andres Freund wrote:
> On 2018-11-26 15:09:43 -0300, Alvaro Herrera wrote:
>> On 2018-Nov-26, Stephen Frost wrote:
>>
>>> I would think we'd have the different GUCs and then the check functions
>>> would only validate that they're valid inputs and then when we get to
>>> the point where we're starting to do recovery we check and make sure
>>> we've been given a sane overall configuration- which means that only
>>> *one* is set, and it matches the recovery target requested.
>>
>> I don't quite understand why it isn't sensible to specify more than one
>> and just stop recovery (or whatever) when at least one of them becomes
>> true.  Maybe I want to terminate just before commit of transaction
>> 12345, but no later than 2018-11-11 12:47 in any case.
> 
> +1

-1. At least for now.

Prior to this patch the last target specified in recovery.conf was the
one used, not a combination of them.

The committed patch did not propose to change that behavior as far as I
can see.  Since there is no way to determine the order of GUCs like
there was for options in recovery.conf, I think it makes sense to
restrict it to a single target type for now.  This is not exactly the
behavior we had before but I think it comes the closest.

Allowing multiple targets could be considered as a separate patch if
anyone is interested.

Regards,
-- 
-David
david@pgmasters.net


pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: pgsql: Integrate recovery.conf into postgresql.conf
Next
From: Dmitry Dolgov
Date:
Subject: Re: Make foo=null a warning by default.