leixm commented on code in PR #4797:
URL: https://github.com/apache/hadoop/pull/4797#discussion_r954657044
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/loghandler/NonAggregatingLogHandler.java:
##########
@@ -209,6 +222,23 @@ public Set<ApplicationId> getInvalidTokenApps() {
return Collections.emptySet();
}
+ private long calculateSizeOfAppLogs(String user, ApplicationId
applicationId) {
+ FileContext lfs = getLocalFileContext(getConfig());
+ long appLogsSize = 0L;
+ for (String rootLogDir : dirsHandler.getLogDirsForCleanup()) {
+ Path logDir = new Path(rootLogDir, applicationId.toString());
+ try {
+ appLogsSize += lfs.getFileStatus(logDir).getLen();
+ } catch (UnsupportedFileSystemException ue) {
+ LOG.warn("Unsupported file system used for log dir " + logDir, ue);
Review Comment:
These codes are not added by me, because CheckStyle doesn't pass, so I
modified the indentation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
