citusdata / citusdata/citus

Re-add connection retries, but less aggressively.

Open
#1,019 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

In the course of #863 connection retries have been removed, because the current logic seems a bit dangerous. What we want is roughly something like:
- perform at most three retries in a single transaction, then fail. That prevents hundreds of retries when e.g. COPY is used, making the problem worse
- add some delay in retrying e.g. connection_timeout / 10 * num_retry or such
- don't retry if the error is due to max connections

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.