Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

YARN-6329. Remove unnecessary TODO comment from AppLogAggregatorImpl.java. Contributed by victor bertschinger. 

(cherry picked from commit 82fb9ce8df)
This commit is contained in:
Akira Ajisaka 2017-03-29 10:35:30 +09:00
parent a65011a128
commit 554bc8342b
No known key found for this signature in database
GPG Key ID: C1EDBB9CA400FD50
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
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
View File

@ -573,8 +573,6 @@ public class AppLogAggregatorImpl implements AppLogAggregator {
(remoteNodeLogFileForApp.getName() + LogAggregationUtils.TMP_FILE_SUFFIX)); (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) { private boolean shouldUploadLogs(ContainerLogContext logContext) {
return logAggPolicy.shouldDoLogAggregation(logContext); return logAggPolicy.shouldDoLogAggregation(logContext);
} }