Re[2]: [HACKERS] Patch: add recovery_timeout option to control timeout of restore_command nonzero status code - Mailing list pgsql-hackers

From Alexey Vasiliev
Subject Re[2]: [HACKERS] Patch: add recovery_timeout option to control timeout of restore_command nonzero status code
Date
Msg-id 1415046923.396879961@f145.i.mail.ru
Whole thread Raw
In response to Re: Patch: add recovery_timeout option to control timeout of restore_command nonzero status code  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: Re[2]: [HACKERS] Patch: add recovery_timeout option to control timeout of restore_command nonzero status code  (Fabrízio de Royes Mello <fabriziomello@gmail.com>)
List pgsql-hackers
Mon, 03 Nov 2014 14:36:33 -0500 от Peter Eisentraut <peter_e@gmx.net>:
> On 11/3/14 6:04 AM, Alexey Vasiliev wrote:
> >  3. What the patch does in a short paragraph: This patch should add
> >     option recovery_timeout, which help to control timeout of
> >     restore_command nonzero status code. Right now default value is 5
> >     seconds. This is useful, if I using for restore of wal logs some
> >     external storage (like AWS S3) and no matter what the slave database
> >     will lag behind the master. The problem, what for each request to
> >     AWS S3 need to pay, what is why for N nodes, which try to get next
> >     wal log each 5 seconds will be bigger price, than for example each
> >     30 seconds.
>
> That seems useful.  I would include something about this use case in the
> documentation.

Ok, I will add this in patch.

>
> > This is my first patch. I am not sure about name of option. Maybe it
> > should called "recovery_nonzero_timeout".
>
> The option name had me confused.  At first I though this is the time
> after which a running restore_command invocation gets killed.  I think a
> more precise description might be restore_command_retry_interval.

"restore_command_retry_interval" - I like this name!

Should I change my patch and send it by email? And also as I understand I should change message ID for
https://commitfest.postgresql.org/action/patch_view?id=1636,isn't it? 

Thanks

--
Alexey Vasiliev

pgsql-hackers by date:

Previous
From: Jim Nasby
Date:
Subject: Re: Let's drop two obsolete features which are bear-traps for novices
Next
From: Alvaro Herrera
Date:
Subject: Re: tracking commit timestamps