hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aengin...@apache.org
Subject [09/50] [abbrv] hadoop git commit: YARN-6329. Remove unnecessary TODO comment from AppLogAggregatorImpl.java. Contributed by victor bertschinger.
Date Tue, 04 Apr 2017 19:56:33 GMT
YARN-6329. Remove unnecessary TODO comment from AppLogAggregatorImpl.java. Contributed by victor
bertschinger.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/82fb9ce8
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/82fb9ce8
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/82fb9ce8

Branch: refs/heads/HDFS-7240
Commit: 82fb9ce8df7a534a4cbcec624d7b6c0b33e79123
Parents: 063b513
Author: Akira Ajisaka <aajisaka@apache.org>
Authored: Wed Mar 29 10:35:30 2017 +0900
Committer: Akira Ajisaka <aajisaka@apache.org>
Committed: Wed Mar 29 10:36:24 2017 +0900

----------------------------------------------------------------------
 .../containermanager/logaggregation/AppLogAggregatorImpl.java      | 2 --
 1 file changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/82fb9ce8/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/logaggregation/AppLogAggregatorImpl.java
----------------------------------------------------------------------
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/logaggregation/AppLogAggregatorImpl.java
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/logaggregation/AppLogAggregatorImpl.java
index 065854e..d70acc9 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/logaggregation/AppLogAggregatorImpl.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/logaggregation/AppLogAggregatorImpl.java
@@ -573,8 +573,6 @@ public class AppLogAggregatorImpl implements AppLogAggregator {
       (remoteNodeLogFileForApp.getName() + LogAggregationUtils.TMP_FILE_SUFFIX));
   }
 
-  // TODO: The condition: containerId.getId() == 1 to determine an AM container
-  // is not always true.
   private boolean shouldUploadLogs(ContainerLogContext logContext) {
     return logAggPolicy.shouldDoLogAggregation(logContext);
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org


Mime
View raw message