On Tue, Jun 25, 2019 at 4:00 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > Fair enough. I have implemented it based on next_retry_at and use > constant time 10s for the next retry. I have used define instead of a > GUC as all the other constants for similar things are defined as of > now. One thing to note is that we want the linger time (defined as > UNDO_WORKER_LINGER_MS) for a undo worker to be more than failure retry > time (defined as UNDO_FAILURE_RETRY_DELAY_MS) as, otherwise, the undo > worker can exit before retrying the failed requests.
Uh, I think we want exactly the opposite. We want the workers to exit before retrying, so that there's a chance for other databases to get processed, I think. Am I confused? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
