Patch: add recovery_timeout option to control timeout of restore_command nonzero status code

Поиск
Список
Период
Сортировка
От Alexey Vasiliev
Тема Patch: add recovery_timeout option to control timeout of restore_command nonzero status code
Дата
Msg-id 1415012640.945585380@f281.i.mail.ru
обсуждение исходный текст
Ответы Re: Patch: add recovery_timeout option to control timeout of restore_command nonzero status code  (Peter Eisentraut <peter_e@gmx.net>)
Re: Patch: add recovery_timeout option to control timeout of restore_command nonzero status code  (Andres Freund <andres@2ndquadrant.com>)
Список pgsql-hackers

Hello everyone.

  1. Project name: Add recovery_timeout option to control timeout of restore_command nonzero status code
  2. Uniquely identifiable file name, so we can tell difference between your v1 and v24: 0001-add-recovery_timeout-to-controll-timeout-between-res.patch
  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. Before I do this in this way: "if ! (/usr/local/bin/envdir /etc/wal-e.d/env /usr/local/bin/wal-e wal-fetch "%f" "%p"); then sleep 60; fi". But in this case restart/stop database slower.
  4. Whether the patch is for discussion or for application: No such thing.
  5. Which branch the patch is against: master branch
  6. Whether it compiles and tests successfully, so we know nothing obvious is broken: compiled and pass tests on local mashine.
  7. Whether it contains any platform-specific items and if so, has it been tested on other platforms: hope, no.
  8. Confirm that the patch includes regression tests to check the new feature actually works as described: No it doesn't have test. I don't found ho to testing new config variables.
  9. Include documentation: added.
  10. Describe the effect your patch has on performance, if any: shouldn't effect on database performance.

This is my first patch. I am not sure about name of option. Maybe it should called "recovery_nonzero_timeout".

--
Alexey Vasiliev
Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Guillaume Lelarge
Дата:
Сообщение: Re: Maximum number of WAL files in the pg_xlog directory
Следующее
От: Fabrízio de Royes Mello
Дата:
Сообщение: Re: CINE in CREATE TABLE AS ... and CREATE MATERIALIZED VIEW ...