Use static ServerVersionNumber to cache the version once

Author: mythz

src/ServiceStack.Redis/RedisNativeClient.cs

@@ -68,8 +68,6 @@ public DateTime? DeactivatedAt
         private IRedisTransactionBase transaction;
         private IRedisPipelineShared pipeline;
 
-        private Dictionary<string, string> info;
-
         const int YES = 1;
         const int NO = 0;
 
@@ -246,21 +244,19 @@ public Dictionary<string, string> Info
         {
             get
             {
-                if (this.info == null)
-                {
-                    var lines = SendExpectString(Commands.Info);
-                    this.info = new Dictionary<string, string>();
+                var lines = SendExpectString(Commands.Info);
+                var info = new Dictionary<string, string>();
 
-                    foreach (var line in lines
-                        .Split(new[] { "\r\n" }, StringSplitOptions.RemoveEmptyEntries))
-                    {
-                        var p = line.IndexOf(':');
-                        if (p == -1) continue;
+                foreach (var line in lines
+                    .Split(new[] { "\r\n" }, StringSplitOptions.RemoveEmptyEntries))
+                {
+                    var p = line.IndexOf(':');
+                    if (p == -1) continue;
 
-                        this.info.Add(line.Substring(0, p), line.Substring(p + 1));
-                    }
+                    info.Add(line.Substring(0, p), line.Substring(p + 1));
                 }
-                return this.info;
+
+                return info;
             }
         }
 

src/ServiceStack.Redis/RedisNativeClient_Utils.cs

@@ -38,7 +38,7 @@ public static int RequestsPerHour
         }
 
         private const int Unknown = -1;
-        public int ServerVersionNumber { get; set; }
+        public static int ServerVersionNumber { get; set; }
 
         public int AssertServerVersionNumber()
         {