Re: Add shutdown_at_recovery_target option to recovery.conf

From: Alex Shulgin <ash(at)commandprompt(dot)com>
To: Christoph Berg <cb(at)df7cb(dot)de>
Cc: Petr Jelinek <petr(at)2ndquadrant(dot)com>, Simon Riggs <simon(at)2ndQuadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add shutdown_at_recovery_target option to recovery.conf
Date: 2014-11-28 16:46:16
Message-ID: 87mw7b5l47.fsf@commandprompt.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Christoph Berg <cb(at)df7cb(dot)de> writes:

> Re: Petr Jelinek 2014-11-25 <5474EFEA(dot)2040000(at)2ndquadrant(dot)com>
>> >Patch committed.

Before I go and rebase that recovery.conf -> GUC patch on top of
this... is it final?

>>
>> Thanks!
>
> I'm a bit late to the party, but wouldn't
>
> recovery_target_action = ...
>
> have been a better name for this? It'd be in line with the other
> recovery_target_* parameters, and also a bit shorter than the imho
> somewhat ugly "action_at_recovery_target".

FWIW, I too think that "recovery_target_action" is a better name.

--
Alex

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Petr Jelinek 2014-11-28 16:59:16 Re: Add shutdown_at_recovery_target option to recovery.conf
Previous Message Peter Eisentraut 2014-11-28 16:43:28 using Core Foundation locale functions