accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bil...@apache.org
Subject git commit: ACCUMULO-1299 removed unneeded hdfs port from test
Date Mon, 23 Sep 2013 17:04:13 GMT
Updated Branches:
  refs/heads/1.5.1-SNAPSHOT 3b773339f -> d7448b2ce


ACCUMULO-1299 removed unneeded hdfs port from test


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/d7448b2c
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/d7448b2c
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/d7448b2c

Branch: refs/heads/1.5.1-SNAPSHOT
Commit: d7448b2ceb626a00e72760c90326b5fc726a853e
Parents: 3b77333
Author: Billie Rinaldi <billie.rinaldi@gmail.com>
Authored: Mon Sep 23 10:03:59 2013 -0700
Committer: Billie Rinaldi <billie.rinaldi@gmail.com>
Committed: Mon Sep 23 10:03:59 2013 -0700

----------------------------------------------------------------------
 .../test/java/org/apache/accumulo/test/AccumuloDFSBase.java   | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/d7448b2c/start/src/test/java/org/apache/accumulo/test/AccumuloDFSBase.java
----------------------------------------------------------------------
diff --git a/start/src/test/java/org/apache/accumulo/test/AccumuloDFSBase.java b/start/src/test/java/org/apache/accumulo/test/AccumuloDFSBase.java
index 333c346..be33e00 100644
--- a/start/src/test/java/org/apache/accumulo/test/AccumuloDFSBase.java
+++ b/start/src/test/java/org/apache/accumulo/test/AccumuloDFSBase.java
@@ -47,11 +47,6 @@ public class AccumuloDFSBase {
   protected static DefaultFileSystemManager vfs = null;
   protected static MiniDFSCluster cluster = null;
   
-  // Choose an IANA unassigned port
-  // http://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.xml
-  // TODO Given that the URI for the NameNode is statically declared in config files in src/test/resources,
we have to assert the given port. It would be better
-  // to let MiniDFSCluster find a free port to use and then update the URI to be used in
the VFS tests - ACCUMULO-1299
-  protected static final Integer HDFS_PORT = 8620;
   protected static final URI HDFS_URI;
   
   static {
@@ -87,7 +82,7 @@ public class AccumuloDFSBase {
     conf.setLong(DFSConfigKeys.DFS_BLOCK_SIZE_KEY, 1024 * 100); // 100K blocksize
     
     try {
-      cluster = new MiniDFSCluster(HDFS_PORT, conf, 1, true, true, true, null, null, null,
null);
+      cluster = new MiniDFSCluster(conf, 1, true, null);
       cluster.waitActive();
       // We can't assume that the hostname of "localhost" will still be "localhost" after
       // starting up the NameNode. We may get mapped into a FQDN via settings in /etc/hosts.


Mime
View raw message