svn commit: r1328594 - /hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java

[ra...@apache.org]

Author: ramkrishna
Date: Sat Apr 21 07:00:46 2012
New Revision: 1328594

URL: http://svn.apache.org/viewvc?rev=1328594&view=rev
Log:
HBASE-5809 Avoid move api to take the destination server same as the source server. Addendum to correct tet case. (Ted)

Modified:
    hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java

Modified: hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java?rev=1328594&r1=1328593&r2=1328594&view=diff
==============================================================================
--- hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java (original)
+++ hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java Sat Apr 21 07:00:46 2012
@@ -655,13 +655,13 @@ public class TestMasterObserver {
     // Try to force a move
     Collection<ServerName> servers = master.getClusterStatus().getServers();
     String destName = null;
-    String firstRegionHostnamePortStr = firstGoodPair.getValue().toString();
-    LOG.info("firstRegionHostnamePortStr=" + firstRegionHostnamePortStr);
+    String serverNameForFirstRegion = firstGoodPair.getValue().toString();
+    LOG.info("serverNameForFirstRegion=" + serverNameForFirstRegion);
     boolean found = false;
     // Find server that is NOT carrying the first region
     for (ServerName info : servers) {
       LOG.info("ServerName=" + info);
-      if (!firstRegionHostnamePortStr.equals(info.getHostAndPort())) {
+      if (!serverNameForFirstRegion.equals(info.getServerName())) {
         destName = info.toString();
         found = true;
         break;