Repository: tez Updated Branches: refs/heads/master cbea5ad3a -> 57c62f1f1
TEZ-2334. ContainerManagementProtocolProxy modifies IPC timeout conf without making a copy. (Chang Li via hitesh) Project: http://git-wip-us.apache.org/repos/asf/tez/repo Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/57c62f1f Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/57c62f1f Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/57c62f1f Branch: refs/heads/master Commit: 57c62f1f1fc9cd22dc858d3d22d7c13cdc427084 Parents: cbea5ad Author: Hitesh Shah <[email protected]> Authored: Fri Apr 17 15:04:02 2015 -0700 Committer: Hitesh Shah <[email protected]> Committed: Fri Apr 17 15:04:02 2015 -0700 ---------------------------------------------------------------------- CHANGES.txt | 1 + .../org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tez/blob/57c62f1f/CHANGES.txt ---------------------------------------------------------------------- diff --git a/CHANGES.txt b/CHANGES.txt index 1888f15..9c86434 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -292,6 +292,7 @@ TEZ-UI CHANGES (TEZ-8): Release 0.5.4: Unreleased ALL CHANGES: + TEZ-2334. ContainerManagementProtocolProxy modifies IPC timeout conf without making a copy. TEZ-2317. Event processing backlog can result in task failures for short tasks TEZ-2289. ATSHistoryLoggingService can generate ArrayOutOfBoundsException. http://git-wip-us.apache.org/repos/asf/tez/blob/57c62f1f/tez-dag/src/main/java/org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java ---------------------------------------------------------------------- diff --git a/tez-dag/src/main/java/org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java b/tez-dag/src/main/java/org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java index cb8e262..a1eb2a7 100644 --- a/tez-dag/src/main/java/org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java +++ b/tez-dag/src/main/java/org/apache/tez/dag/app/launcher/ContainerLauncherImpl.java @@ -259,8 +259,9 @@ public class ContainerLauncherImpl extends AbstractService implements @Override public void serviceStart() { + // pass a copy of config to ContainerManagementProtocolProxy until YARN-3497 is fixed cmProxy = - new ContainerManagementProtocolProxy(getConfig()); + new ContainerManagementProtocolProxy(new Configuration(getConfig())); ThreadFactory tf = new ThreadFactoryBuilder().setNameFormat( "ContainerLauncher #%d").setDaemon(true).build();
