Re: Add a hint when postgresql.conf hot_standby = 'off' and recovery.conf standby = 'on'

From: "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>
To: Josh Berkus <josh(at)agliodbs(dot)com>, hlinnaka(at)iki(dot)fi, Matthew Kelly <mkelly(at)tripadvisor(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add a hint when postgresql.conf hot_standby = 'off' and recovery.conf standby = 'on'
Date: 2015-05-22 18:22:37
Message-ID: 555F73ED.1000203@commandprompt.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 05/22/2015 11:01 AM, Josh Berkus wrote:
>
> On 05/22/2015 12:22 AM, Heikki Linnakangas wrote:
>>> It seems worth adding a hint and/or changing the error message to be
>>> more descriptive when in this state. Any options about what should
>>> be logged before I start putting together a patch?
>>
>> Yeah, might be worthwhile. Perhaps:
>>
>> FATAL: the database system is in standby mode and hot_standby is not
>> enabled
>>
>> Or just:
>>
>> FATAL: the database system is in cold standby mode
>
> Warm Standby is what we called it in the past, so I think we should be
> consistent. Otherwise +1.

As I recall, warm standby is actually what it is. A cold standby is not
applying logs. A warm standby is. So if there is a recovery.conf and
more importantly PostgreSQL is running, it is going to be a warm standby
not a cold.

Other than that, absolute +1.

JD

--
Command Prompt, Inc. - http://www.commandprompt.com/ 503-667-4564
PostgreSQL Centered full stack support, consulting and development.
Announcing "I'm offended" is basically telling the world you can't
control your own emotions, so everyone else should do it for you.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim Nasby 2015-05-22 18:28:35 Re: Change pg_cancel_*() to ignore current backend
Previous Message Jim Nasby 2015-05-22 18:11:42 Re: RFC: Non-user-resettable SET SESSION AUTHORISATION