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 br...@apache.org on 2016/12/31 10:49:12 UTC
[2/2] hadoop git commit: HADOOP-13883. Addendum patch to correct the
message and alphabetized with the earlier patch. Contributed by Yiqun Lin.
HADOOP-13883. Addendum patch to correct the message and alphabetized with the earlier patch. Contributed by Yiqun Lin.
(cherry picked from commit 6938b677111a5fa6bf654717b310220f98b7cf20)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/d6e5e428
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/d6e5e428
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/d6e5e428
Branch: refs/heads/branch-2
Commit: d6e5e4288ca35902a2257fa3862b5e1342903cad
Parents: 1bc9b31
Author: Brahma Reddy Battula <br...@apache.org>
Authored: Sat Dec 31 16:16:02 2016 +0530
Committer: Brahma Reddy Battula <br...@apache.org>
Committed: Sat Dec 31 16:17:15 2016 +0530
----------------------------------------------------------------------
.../org/apache/hadoop/util/GenericOptionsParser.java | 14 ++++++++------
.../hadoop-common/src/site/markdown/CommandsManual.md | 2 +-
.../hadoop/mapred/pipes/TestPipeApplication.java | 4 +++-
3 files changed, 12 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/d6e5e428/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/GenericOptionsParser.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/GenericOptionsParser.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/GenericOptionsParser.java
index 04e8024..f01bcbb 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/GenericOptionsParser.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/GenericOptionsParser.java
@@ -212,10 +212,11 @@ public class GenericOptionsParser {
*/
@SuppressWarnings("static-access")
private static synchronized Options buildGeneralOptions(Options opts) {
- Option fs = OptionBuilder.withArgName("local|namenode:port")
- .hasArg()
- .withDescription("specify a namenode")
- .create("fs");
+ Option fs = OptionBuilder.withArgName("file:///|hdfs://namenode:port")
+ .hasArg()
+ .withDescription("specify default filesystem URL to use, "
+ + "overrides 'fs.defaultFS' property from configurations.")
+ .create("fs");
Option jt = OptionBuilder.withArgName("local|resourcemanager:port")
.hasArg()
.withDescription("specify a ResourceManager")
@@ -569,8 +570,9 @@ public class GenericOptionsParser {
+ "specify an application configuration file");
out.println("-D <property=value> "
+ "define a value for a given property");
- out.println("-fs <local|namenode:port> "
- + "specify a namenode");
+ out.println("-fs <file:///|hdfs://namenode:port> "
+ + "specify default filesystem URL to use, overrides "
+ + "'fs.defaultFS' property from configurations.");
out.println("-jt <local|resourcemanager:port> "
+ "specify a ResourceManager");
out.println("-files <file1,...> "
http://git-wip-us.apache.org/repos/asf/hadoop/blob/d6e5e428/hadoop-common-project/hadoop-common/src/site/markdown/CommandsManual.md
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/site/markdown/CommandsManual.md b/hadoop-common-project/hadoop-common/src/site/markdown/CommandsManual.md
index cf6521e..03dccb9 100644
--- a/hadoop-common-project/hadoop-common/src/site/markdown/CommandsManual.md
+++ b/hadoop-common-project/hadoop-common/src/site/markdown/CommandsManual.md
@@ -39,11 +39,11 @@ Many subcommands honor a common set of configuration options to alter their beha
| GENERIC\_OPTION | Description |
|:---- |:---- |
-| `-fs <local> or <namenode:port>` | Specify a NameNode. |
| `-archives <comma separated list of archives> ` | Specify comma separated archives to be unarchived on the compute machines. Applies only to job. |
| `-conf <configuration file> ` | Specify an application configuration file. |
| `-D <property>=<value> ` | Use value for given property. |
| `-files <comma separated list of files> ` | Specify comma separated files to be copied to the map reduce cluster. Applies only to job. |
+| `-fs <file:///> or <hdfs://namenode:port>` | Specify default filesystem URL to use. Overrides 'fs.defaultFS' property from configurations. |
| `-jt <local> or <resourcemanager:port>` | Specify a ResourceManager. Applies only to job. |
| `-libjars <comma seperated list of jars> ` | Specify comma separated jar files to include in the classpath. Applies only to job. |
http://git-wip-us.apache.org/repos/asf/hadoop/blob/d6e5e428/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/pipes/TestPipeApplication.java
----------------------------------------------------------------------
diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/pipes/TestPipeApplication.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/pipes/TestPipeApplication.java
index 5fbce5d..9b88c5c 100644
--- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/pipes/TestPipeApplication.java
+++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/pipes/TestPipeApplication.java
@@ -306,7 +306,9 @@ public class TestPipeApplication {
"-D <property=value> define a value for a given "
+ "property"));
assertTrue(out.toString()
- .contains("-fs <local|namenode:port> specify a namenode"));
+ .contains("-fs <file:///|hdfs://namenode:port> "
+ + "specify default filesystem URL to use, overrides "
+ + "'fs.defaultFS' property from configurations."));
assertTrue(out.toString().contains(
"-jt <local|resourcemanager:port> specify a ResourceManager"));
assertTrue(out.toString().contains(
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org