YARN-2769. Fixed the problem that timeline domain is not set in distributed shell AM when using shell_command on Windows. Contributed by Varun Vasudev.
This commit is contained in:
parent
5c900b522e
commit
a8c1202220
|
@ -795,6 +795,9 @@ Release 2.6.0 - UNRELEASED
|
||||||
YARN-2747. Fixed the test failure of TestAggregatedLogFormat when native I/O is
|
YARN-2747. Fixed the test failure of TestAggregatedLogFormat when native I/O is
|
||||||
enabled. (Xuan Gong via zjshen)
|
enabled. (Xuan Gong via zjshen)
|
||||||
|
|
||||||
|
YARN-2769. Fixed the problem that timeline domain is not set in distributed shell
|
||||||
|
AM when using shell_command on Windows. (Varun Vasudev via zjshen)
|
||||||
|
|
||||||
Release 2.5.1 - 2014-09-05
|
Release 2.5.1 - 2014-09-05
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
|
@ -470,9 +470,6 @@ public class ApplicationMaster {
|
||||||
shellScriptPathLen = Long.valueOf(envs
|
shellScriptPathLen = Long.valueOf(envs
|
||||||
.get(DSConstants.DISTRIBUTEDSHELLSCRIPTLEN));
|
.get(DSConstants.DISTRIBUTEDSHELLSCRIPTLEN));
|
||||||
}
|
}
|
||||||
if (envs.containsKey(DSConstants.DISTRIBUTEDSHELLTIMELINEDOMAIN)) {
|
|
||||||
domainId = envs.get(DSConstants.DISTRIBUTEDSHELLTIMELINEDOMAIN);
|
|
||||||
}
|
|
||||||
if (!scriptPath.isEmpty()
|
if (!scriptPath.isEmpty()
|
||||||
&& (shellScriptPathTimestamp <= 0 || shellScriptPathLen <= 0)) {
|
&& (shellScriptPathTimestamp <= 0 || shellScriptPathLen <= 0)) {
|
||||||
LOG.error("Illegal values in env for shell script path" + ", path="
|
LOG.error("Illegal values in env for shell script path" + ", path="
|
||||||
|
@ -483,6 +480,10 @@ public class ApplicationMaster {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (envs.containsKey(DSConstants.DISTRIBUTEDSHELLTIMELINEDOMAIN)) {
|
||||||
|
domainId = envs.get(DSConstants.DISTRIBUTEDSHELLTIMELINEDOMAIN);
|
||||||
|
}
|
||||||
|
|
||||||
containerMemory = Integer.parseInt(cliParser.getOptionValue(
|
containerMemory = Integer.parseInt(cliParser.getOptionValue(
|
||||||
"container_memory", "10"));
|
"container_memory", "10"));
|
||||||
containerVirtualCores = Integer.parseInt(cliParser.getOptionValue(
|
containerVirtualCores = Integer.parseInt(cliParser.getOptionValue(
|
||||||
|
|
Loading…
Reference in New Issue