Merge pull request #246 from cnblogs/fix-node-locator-setting

fix: set UseLegacyNodeLocator before SetNodeLocator
cnblogs · Dec 9, 2024 · d25f5ec · d25f5ec
2 parents b25aabc + 0e26ec0
commit d25f5ec
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/src/Enyim.Caching/Configuration/MemcachedClientConfiguration.cs b/src/Enyim.Caching/Configuration/MemcachedClientConfiguration.cs
@@ -60,7 +60,12 @@ public MemcachedClientConfiguration(
             }
 #endif
 
+            _useLegacyNodeLocator = options.UseLegacyNodeLocator;
             ConfigureServers(options);
+            if (NodeLocator == null)
+            {
+                SetNodeLocator();
+            }
 
             SocketPool = new SocketPoolConfiguration();
             if (options.SocketPool != null)
@@ -155,12 +160,6 @@ public MemcachedClientConfiguration(
                 _logger.LogDebug($"Use KeyTransformer Type : '{keyTransformer}'");
             }
 
-            _useLegacyNodeLocator = options.UseLegacyNodeLocator;
-            if (NodeLocator == null)
-            {
-                SetNodeLocator();
-            }
-
             if (!string.IsNullOrEmpty(options.Transcoder))
             {
                 try
@@ -234,7 +233,7 @@ private void SetNodeLocator()
                 NodeLocator = typeof(SingleNodeLocator);
             }
 
-            _logger.LogDebug("Use NodeLocator: {NodeLocator}", NodeLocator);
+            _logger.LogDebug("Use NodeLocator: {NodeLocator}. Current server count: {serverCount}", NodeLocator, Servers.Count);
         }
 
         /// <summary>