Fix bug in clientMode

Author: YueTang-Vanessa

src/main/java/net/spy/memcached/MemcachedClient.java

@@ -248,6 +248,7 @@ private MemcachedClient(ConnectionFactory cf, List<InetSocketAddress> addrs, boo
     //An internal customer convenience check to determine whether the client mode based on 
     // the DNS name if only one endpoint is specified. 
     if(determineClientMode){
+      boolean isClientModeDetermined = false;
       if(addrs.size() == 1){
         if(addrs.get(0) == null){
           throw new NullPointerException("Socket address is null");
@@ -256,10 +257,14 @@ private MemcachedClient(ConnectionFactory cf, List<InetSocketAddress> addrs, boo
         //All config endpoints has ".cfg." subdomain in the DNS name.
         if(hostName != null && hostName.contains(".cfg.")){
           cf = updateClientMode(cf, ClientMode.Dynamic);
+          isClientModeDetermined = true;
         }
       }
       //Fallback to static mode
-      cf = updateClientMode(cf, ClientMode.Static);
+      if (!isClientModeDetermined) {
+        cf = updateClientMode(cf, ClientMode.Static);
+        isClientModeDetermined = true;
+      }
     }
 
     if (cf == null) {

src/test/java/net/spy/memcached/MemcachedConnectionTest.java

@@ -73,10 +73,14 @@ public void testBelongsToCluster() throws Exception {
     assertTrue(conn.belongsToCluster(node));
     assertFalse(conn.belongsToCluster(node2));
   }
-  public void testDefaultClientMode() throws Exception {
+
+  public void testClientMode() throws Exception {
     ConnectionFactory factory = new DefaultConnectionFactory();
     assert factory.getClientMode() == ClientMode.Unset;
-    MemcachedClient client = new MemcachedClient(factory, AddrUtil.getAddresses(UnitTestConfig.IPV4_ADDR + ":11211"));
-    assert client.clientMode == ClientMode.Static;
+    MemcachedClient client1 = new MemcachedClient(factory, AddrUtil.getAddresses(UnitTestConfig.IPV4_ADDR + ":11211"));
+    assert client1.clientMode == ClientMode.Static;
+    MemcachedClient client2 = new MemcachedClient(AddrUtil.getAddresses(UnitTestConfig.TEST_HOSTNAME + ":11211"));
+    assert client2.clientMode == ClientMode.Dynamic;
   }
+
 }

src/test/java/net/spy/memcached/UnitTestConfig.java

@@ -36,6 +36,7 @@ public final class UnitTestConfig {
   public static final String PORT_PROP = "server.port_number";
   public static final String TYPE_TEST_UNIT = "unit";
   public static final String TYPE_TEST_CI = "ci";
+  public static final String TEST_HOSTNAME = "abc.cfg.test.com";
 
   //currently server host address ipv4 is always default to "127.0.0.1", disabled in build.xml
   public static final String IPV4_ADDR = System.getProperty(IPV4_PROP,