Merge pull request #32 from Cimpress-MCP/fix_redisHostIndex_inc

Move redisHostIndex inc to a finally
2015-03-03 13:01:02 -05:00
parent cdc2d09150 42741fbe1e
commit fd123e3a86
1 changed files with 6 additions and 5 deletions
--- a/TimberWinR/Outputs/Redis.cs
+++ b/TimberWinR/Outputs/Redis.cs
@@ -60,16 +60,17 @@ namespace TimberWinR.Outputs
                try
                {
                    RedisClient client = new RedisClient(_redisHosts[_redisHostIndex], _port, _timeout);
-
-                    _redisHostIndex++;
-                    if (_redisHostIndex >= _redisHosts.Length)
-                        _redisHostIndex = 0;
-
                    return client;
                }
                catch (Exception)
                {
                }
+                finally
+                {
+                    _redisHostIndex++;
+                    if (_redisHostIndex >= _redisHosts.Length)
+                        _redisHostIndex = 0;
+                }
                numTries++;
            }