Patch: add recovery_timeout option to control timeout of restore_command nonzero status code - Mailing list pgsql-hackers

From Alexey Vasiliev
Subject Patch: add recovery_timeout option to control timeout of restore_command nonzero status code
Date
Msg-id 1415012640.945585380@f281.i.mail.ru
Whole thread Raw
Responses 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>)
List 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
Attachment

pgsql-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Maximum number of WAL files in the pg_xlog directory
Next
From: Fabrízio de Royes Mello
Date:
Subject: Re: CINE in CREATE TABLE AS ... and CREATE MATERIALIZED VIEW ...