fix host index to look at Host.Count instead of client.Length

mythz · mythz · commit c94eedd2e346 · 2014-12-10T18:05:21.000-05:00
diff --git a/src/ServiceStack.Redis/RedisManagerPool.cs b/src/ServiceStack.Redis/RedisManagerPool.cs
@@ -123,7 +123,7 @@ public IRedisClient GetClient()
                 while ((inActiveClient = GetInActiveClient()) == null)
                 {
                     //Create new client outside of pool when max pool size exceeded
-                    var nextIndex = poolIndex % clients.Length;
+                    var nextIndex = poolIndex % Hosts.Count;
                     var nextHost = Hosts[nextIndex];
                     var newClient = InitNewClient(nextHost);
                     //Don't handle callbacks for new client outside pool

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ public IRedisClient GetClient()`
`123`	`123`	`while ((inActiveClient = GetInActiveClient()) == null)`
`124`	`124`	`{`
`125`	`125`	`//Create new client outside of pool when max pool size exceeded`
`126`		`- var nextIndex = poolIndex % clients.Length;`
	`126`	`+ var nextIndex = poolIndex % Hosts.Count;`
`127`	`127`	`var nextHost = Hosts[nextIndex];`
`128`	`128`	`var newClient = InitNewClient(nextHost);`
`129`	`129`	`//Don't handle callbacks for new client outside pool`