diff options
author | Harald Musum <musum@yahooinc.com> | 2021-10-05 15:08:25 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2021-10-05 15:08:25 +0200 |
commit | 060961ce51b699ca575a1a51fe87787b8fedf79b (patch) | |
tree | 708178b1633ccb1adaee948193b9c49cf6662d1e /config-proxy | |
parent | f9b6a26a7872f06ec98166b3b610a16ebd0764e6 (diff) |
Simplify backoff when getting config fails
Use just one way of calculating delay until sending next request after and error:
* do not retry faster after transient error, it may be overload
* backoff based on number of failures
* Use a max delay, but use a random factor anyway
* Simplify
Diffstat (limited to 'config-proxy')
-rw-r--r-- | config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java index a2373111bac..2e8685887c6 100644 --- a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java +++ b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java @@ -259,8 +259,7 @@ class RpcConfigSourceClient implements ConfigSourceClient, Runnable { // Proxy should time out before clients upon subscription. double timingValuesRatio = 0.8; TimingValues tv = new TimingValues(); - tv.setUnconfiguredDelay((long) (tv.getUnconfiguredDelay() * timingValuesRatio)). - setConfiguredErrorDelay((long) (tv.getConfiguredErrorDelay() * timingValuesRatio)). + tv.setFixedDelay((long) (tv.getFixedDelay() * timingValuesRatio)). setSubscribeTimeout((long) (tv.getSubscribeTimeout() * timingValuesRatio)). setConfiguredErrorTimeout(-1); // Never cache errors return tv; |