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 8795e0b TS-4341 correct warning check
8795e0b is described below
commit 8795e0b0ddb66073a1948bb1309e007fe63bc323
Author: Thomas Jackson <[email protected]>
AuthorDate: Fri Apr 15 11:56:14 2016 -0700
TS-4341 correct warning check
we should only warn users if the queue is enabled-- which is a non-negative
number.
---
proxy/http/HttpConfig.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/proxy/http/HttpConfig.cc b/proxy/http/HttpConfig.cc
index 1ee7ed5..504a385 100644
--- a/proxy/http/HttpConfig.cc
+++ b/proxy/http/HttpConfig.cc
@@ -1165,7 +1165,7 @@ HttpConfig::reconfigure()
params->oride.origin_max_connections = m_master.oride.origin_max_connections;
params->oride.origin_max_connections_queue =
m_master.oride.origin_max_connections_queue;
// if origin_max_connections_queue is set without max_connections, it is
meaningless, so we'll warn
- if (params->oride.origin_max_connections_queue &&
+ if (params->oride.origin_max_connections_queue >= 0 &&
!(params->oride.origin_max_connections ||
params->origin_min_keep_alive_connections)) {
Warning("origin_max_connections_queue is set, but neither
origin_max_connections nor origin_min_keep_alive_connections are "
"set, please correct your records.config");
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
