You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ja...@apache.org on 2016/10/27 00:13:20 UTC
[trafficserver] branch master updated: TS-4968: Log a warning if
connect_attempts_rr_retries is >= connect_attempts_max_retries
This is an automated email from the ASF dual-hosted git repository.
jacksontj pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new bc84981 TS-4968: Log a warning if connect_attempts_rr_retries is >= connect_attempts_max_retries
bc84981 is described below
commit bc849813fc49986c5a600a24ce74cd2aa8c2bad8
Author: Thomas Jackson <ja...@gmail.com>
AuthorDate: Thu Oct 13 11:53:20 2016 -0700
TS-4968: Log a warning if connect_attempts_rr_retries is >= connect_attempts_max_retries
---
proxy/http/HttpConfig.cc | 18 ++++++++++++------
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/proxy/http/HttpConfig.cc b/proxy/http/HttpConfig.cc
index 8936ff5..45aa653 100644
--- a/proxy/http/HttpConfig.cc
+++ b/proxy/http/HttpConfig.cc
@@ -1224,12 +1224,18 @@ HttpConfig::reconfigure()
params->oride.connect_attempts_max_retries = m_master.oride.connect_attempts_max_retries;
params->oride.connect_attempts_max_retries_dead_server = m_master.oride.connect_attempts_max_retries_dead_server;
- params->oride.connect_attempts_rr_retries = m_master.oride.connect_attempts_rr_retries;
- params->oride.connect_attempts_timeout = m_master.oride.connect_attempts_timeout;
- params->oride.post_connect_attempts_timeout = m_master.oride.post_connect_attempts_timeout;
- params->oride.parent_connect_attempts = m_master.oride.parent_connect_attempts;
- params->per_parent_connect_attempts = m_master.per_parent_connect_attempts;
- params->parent_connect_timeout = m_master.parent_connect_timeout;
+ if (m_master.oride.connect_attempts_rr_retries >= params->oride.connect_attempts_max_retries) {
+ Warning("connect_attempts_rr_retries (%d) is greater than connect_attempts_max_retries (%d), this means requests will never "
+ "redispatch "
+ "to another server",
+ m_master.oride.connect_attempts_rr_retries, params->oride.connect_attempts_max_retries);
+ }
+ params->oride.connect_attempts_rr_retries = m_master.oride.connect_attempts_rr_retries;
+ params->oride.connect_attempts_timeout = m_master.oride.connect_attempts_timeout;
+ params->oride.post_connect_attempts_timeout = m_master.oride.post_connect_attempts_timeout;
+ params->oride.parent_connect_attempts = m_master.oride.parent_connect_attempts;
+ params->per_parent_connect_attempts = m_master.per_parent_connect_attempts;
+ params->parent_connect_timeout = m_master.parent_connect_timeout;
params->oride.sock_recv_buffer_size_out = m_master.oride.sock_recv_buffer_size_out;
params->oride.sock_send_buffer_size_out = m_master.oride.sock_send_buffer_size_out;
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].