Re: pgsql: action_at_recovery_target recovery config option

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: action_at_recovery_target recovery config option
Date: 2014-11-26 12:37:51
Message-ID: CA+U5nM+Oetd_h-onF2FOAWf2Td=BZzkQDNFk0MoP4ZHtPohejg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 26 November 2014 at 00:38, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>
>
> On Wed, Nov 26, 2014 at 7:33 AM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
>> On 25 November 2014 at 21:40, Stefan Kaltenbrunner
>> <stefan(at)kaltenbrunner(dot)cc> wrote:
>>> On 11/25/2014 09:14 PM, Simon Riggs wrote:
>>>> action_at_recovery_target recovery config option
>>>>
>>>> action_at_recovery_target = pause | promote | shutdown
>>>>
>>>> Petr Jelinek
>>>>
>>>> Reviewed by Muhammad Asif Naeem, Fujji Masao and
>>>> Simon Riggs
>>>>
>>>> Branch
>>>> ------
>>>> master
>>>>
>>>> Details
>>>> -------
>>>>
>>>> http://git.postgresql.org/pg/commitdiff/aedccb1f6fef988af1d1a25b78151f3773954b4c
>>>>
>>>> Modified Files
>>>> --------------
>>>> doc/src/sgml/recovery-config.sgml | 58 ++++++++++++++++++---
>>>> src/backend/access/transam/xlog.c | 98
>>>> +++++++++++++++++++++++++++++++----
>>>> src/backend/postmaster/postmaster.c | 12 +++++
>>>> src/include/access/xlog_internal.h | 10 ++++
>>>> 4 files changed, 159 insertions(+), 19 deletions(-)
>>>
>>>
>>> looks like this broke the docs build:
>>>
>>>
>>> http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=guaibasaurus&dt=2014-11-25%2020%3A17%3A01
>>
>> Thanks, hopefully fixed soon afterwards.
> That's still broken even after you pushed b7a7158 (blind commit?) as a
> tentative to fix it:
> openjade:recovery-config.sgml:319:8:E: character data is not allowed here
> openjade:recovery-config.sgml:319:36:E: document type does not allow element
> "LITERAL" here; missing one of "REMARK", "SYNOPSIS", "LITERALLAYOUT",
> "PROGRAMLISTING", "SCREEN", "PARA", "SIMPARA", "BRIDGEHEAD" start-tag
> openjade:recovery-config.sgml:319:46:E: character data is not allowed here
> openjade:recovery-config.sgml:322:23:E: document type does not allow element
> "FUNCTION" here; missing one of "REMARK", "SYNOPSIS", "LITERALLAYOUT",
> "PROGRAMLISTING", "SCREEN", "PARA", "SIMPARA", "BRIDGEHEAD" start-tag
> openjade:recovery-config.sgml:322:51:E: character data is not allowed here
> openjade:recovery-config.sgml:323:56:E: document type does not allow element
> "XREF" here; missing one of "REMARK", "SYNOPSIS", "LITERALLAYOUT",
> "PROGRAMLISTING", "SCREEN", "PARA", "SIMPARA", "BRIDGEHEAD" start-tag
> openjade:recovery-config.sgml:328:13:E: end tag for element "PARA" which is
> not open
>
> The docs are able to compile correctly with the patch attached, you simply
> missed a <para>.
> Thanks,

Thanks, will apply.

--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2014-11-26 12:43:33 Re: pgsql: action_at_recovery_target recovery config option
Previous Message Heikki Linnakangas 2014-11-26 09:07:15 pgsql: Remove dead function prototype