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

From: Josh Berkus <josh(at)agliodbs(dot)com>
To: 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:01:05
Message-ID: 555F6EE1.50901@agliodbs.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

The additional benefit of this is that it would (hopefully) allow us to
distinguish between a warm standby which was still reading its own xlogs
(i.e. in crash recovery) and a warm standby which was using the
restore_command (i.e. standby-ing). For that reason, it would be ideal
if the new message only displayed once the restore_command starts being
used.

That is:
Cold Standby == DB Snapshot and a huge folder of WAL files (i.e. Barman)
Warm Standby == hot_standby=off, recoveryconf.standby=on
Hot Standby == hot_standby=on, recoveryconf.standby=on

--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2015-05-22 18:08:07 Re: [COMMITTERS] pgsql: Row-Level Security Policies (RLS)
Previous Message Andres Freund 2015-05-22 17:50:21 Re: Issues in Replication Progress Tracking