tez-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ss...@apache.org
Subject [17/50] [abbrv] tez git commit: TEZ-2334. ContainerManagementProtocolProxy modifies IPC timeout conf without making a copy. (Chang Li via hitesh)
Date Fri, 24 Apr 2015 00:26:11 GMT
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/TEZ-2003
Commit: 57c62f1f1fc9cd22dc858d3d22d7c13cdc427084
Parents: cbea5ad
Author: Hitesh Shah <hitesh@apache.org>
Authored: Fri Apr 17 15:04:02 2015 -0700
Committer: Hitesh Shah <hitesh@apache.org>
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();


Mime
View raw message