From: | Alexey Vasiliev <leopard_ne(at)inbox(dot)ru> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re[2]: [HACKERS] Patch: add recovery_timeout option to control timeout of restore_command nonzero status code |
Date: | 2014-11-03 20:35:23 |
Message-ID: | 1415046923.396879961@f145.i.mail.ru |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Mon, 03 Nov 2014 14:36:33 -0500 от Peter Eisentraut <peter_e(at)gmx(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2014-11-03 20:36:48 | Re: tracking commit timestamps |
Previous Message | Jim Nasby | 2014-11-03 20:29:18 | Re: Let's drop two obsolete features which are bear-traps for novices |