]> git.kaiwu.me - haproxy.git/commit
BUG/MINOR: use the same check condition for server as other algorithms
authorGodbach <nylzhaowei@gmail.com>
Wed, 7 Aug 2013 01:48:23 +0000 (09:48 +0800)
committerWilly Tarreau <w@1wt.eu>
Tue, 13 Aug 2013 14:15:08 +0000 (16:15 +0200)
commit8f9fd2f0a0893761afeb6800c7b62a51d782af0e
tree874d94dd7ebaae7ed930ea68a721e118bf881fcf
parent00f0084752eab236af80e61291d672e835790cff
BUG/MINOR: use the same check condition for server as other algorithms

Such load balance algorithms as roundrobin, leastconn and first will check the
server after being selected with the following condition:
if (!s->maxconn || (!s->nbpend && s->served < srv_dynamic_maxconn(s)))

But static-rr uses the different one in map_get_server_rr()  as below:
if (!srv->maxconn || srv->cur_sess < srv_dynamic_maxconn(srv))
After viewing this difference, it is a better choice for static-rr to use the
same check condition as other algorithms.

This change will only affect static-rr. Though all hash algorithms with type
map-based will use the same server map as static-rr, they call another function
map_get_server_hash() to get server.

Signed-off-by: Godbach <nylzhaowei@gmail.com>
src/lb_map.c