You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by wh...@apache.org on 2015/02/19 00:14:57 UTC
hadoop git commit: HDFS-7808. Remove obsolete -ns options in in
DFSHAAdmin.java. Contributed by Arshad Mohammad.
Repository: hadoop
Updated Branches:
refs/heads/branch-2 b1b1e05e1 -> 3605928bd
HDFS-7808. Remove obsolete -ns options in in DFSHAAdmin.java. Contributed by Arshad Mohammad.
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/3605928b
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/3605928b
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/3605928b
Branch: refs/heads/branch-2
Commit: 3605928bd090285bd037832953d03c2d3d2a2d44
Parents: b1b1e05
Author: Haohui Mai <wh...@apache.org>
Authored: Wed Feb 18 15:14:39 2015 -0800
Committer: Haohui Mai <wh...@apache.org>
Committed: Wed Feb 18 15:14:53 2015 -0800
----------------------------------------------------------------------
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++
.../apache/hadoop/hdfs/tools/DFSHAAdmin.java | 20 --------------------
.../hadoop/hdfs/tools/TestDFSHAAdmin.java | 20 --------------------
.../hdfs/tools/TestDFSHAAdminMiniCluster.java | 3 ---
4 files changed, 3 insertions(+), 43 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/3605928b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
index 3fb82c6..9c152d9 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -682,6 +682,9 @@ Release 2.7.0 - UNRELEASED
HDFS-6662. WebHDFS cannot open a file if its path contains "%".
(Gerson Carlos via wheat9)
+ HDFS-7808. Remove obsolete -ns options in in DFSHAAdmin.java.
+ (Arshad Mohammad via wheat9)
+
BREAKDOWN OF HDFS-7584 SUBTASKS AND RELATED JIRAS
HDFS-7720. Quota by Storage Type API, tools and ClientNameNode
http://git-wip-us.apache.org/repos/asf/hadoop/blob/3605928b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
index 1ec6d35..6b6fb30 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
@@ -18,7 +18,6 @@
package org.apache.hadoop.hdfs.tools;
import java.io.PrintStream;
-import java.util.Arrays;
import java.util.Collection;
import org.apache.commons.logging.Log;
@@ -98,25 +97,6 @@ public class DFSHAAdmin extends HAAdmin {
printUsage(errOut);
return -1;
}
-
- int i = 0;
- String cmd = argv[i++];
-
- if ("-ns".equals(cmd)) {
- if (i == argv.length) {
- errOut.println("Missing nameservice ID");
- printUsage(errOut);
- return -1;
- }
- nameserviceId = argv[i++];
- if (i >= argv.length) {
- errOut.println("Missing command");
- printUsage(errOut);
- return -1;
- }
- argv = Arrays.copyOfRange(argv, i, argv.length);
- }
-
return super.runCmd(argv);
}
http://git-wip-us.apache.org/repos/asf/hadoop/blob/3605928b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
index 33da4d4..8ecc71a 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
@@ -147,17 +147,6 @@ public class TestDFSHAAdmin {
}
@Test
- public void testNameserviceOption() throws Exception {
- assertEquals(-1, runTool("-ns"));
- assertOutputContains("Missing nameservice ID");
- assertEquals(-1, runTool("-ns", "ns1"));
- assertOutputContains("Missing command");
- // "ns1" isn't defined but we check this lazily and help doesn't use the ns
- assertEquals(0, runTool("-ns", "ns1", "-help", "transitionToActive"));
- assertOutputContains("Transitions the service into Active");
- }
-
- @Test
public void testNamenodeResolution() throws Exception {
Mockito.doReturn(STANDBY_READY_RESULT).when(mockProtocol).getServiceStatus();
assertEquals(0, runTool("-getServiceState", "nn1"));
@@ -279,15 +268,6 @@ public class TestDFSHAAdmin {
}
@Test
- public void testFailoverWithFencerAndNameservice() throws Exception {
- Mockito.doReturn(STANDBY_READY_RESULT).when(mockProtocol).getServiceStatus();
- HdfsConfiguration conf = getHAConf();
- conf.set(DFSConfigKeys.DFS_HA_FENCE_METHODS_KEY, getFencerTrueCommand());
- tool.setConf(conf);
- assertEquals(0, runTool("-ns", "ns1", "-failover", "nn1", "nn2"));
- }
-
- @Test
public void testFailoverWithFencerConfiguredAndForce() throws Exception {
Mockito.doReturn(STANDBY_READY_RESULT).when(mockProtocol).getServiceStatus();
HdfsConfiguration conf = getHAConf();
http://git-wip-us.apache.org/repos/asf/hadoop/blob/3605928b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
index ee1c184..b4d45c3 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
@@ -156,9 +156,6 @@ public class TestDFSHAAdminMiniCluster {
assertEquals(0, runTool("-transitionToActive", "nn1"));
assertEquals(0, runTool("-failover", "nn1", "nn2"));
- // Test failover with fencer and nameservice
- assertEquals(0, runTool("-ns", "minidfs-ns", "-failover", "nn2", "nn1"));
-
// Fencer has not run yet, since none of the above required fencing
assertEquals("", Files.toString(tmpFile, Charsets.UTF_8));