svn commit: r1181591 - /hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java

[ns...@apache.org]

Author: nspiegelberg
Date: Tue Oct 11 02:23:22 2011
New Revision: 1181591

URL: http://svn.apache.org/viewvc?rev=1181591&view=rev
Log:
TestReplicationSourceManager fails due to port conflict

Summary: If mcproxy is running on local box, this unit test often fails because
of port conflict on 60020.
Test Plan: ran unit test with mcproxy running before and after fix to confirm
the test now passes ok.
Reviewed By: mbautin
Reviewers: mbautin, nspiegelberg
CC: hbase@lists, mbautin, kannan
Differential Revision: 277177

Modified:
    hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java

Modified: hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java?rev=1181591&r1=1181590&r2=1181591&view=diff
==============================================================================
--- hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java (original)
+++ hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java Tue Oct 11 02:23:22 2011
@@ -107,6 +107,10 @@ public class TestReplicationSourceManage
           conf.get(HConstants.ZOOKEEPER_QUORUM)+":" +
           conf.get("hbase.zookeeper.property.clientPort")+":/1");
 
+    // set port to 0 so that RS picks a port dynamically to
+    // avoid clash over the default port in unit test runs.
+    conf.set(HConstants.REGIONSERVER_PORT, "0");
+
     HRegionServer server = new HRegionServer(conf);
     ReplicationZookeeperWrapper helper = new ReplicationZookeeperWrapper(
         server.getZooKeeperWrapper(), conf,