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

YARN-3237. AppLogAggregatorImpl fails to log error cause. Contributed by 

Rushabh S Shah

(cherry picked from commit f56c65bb3e)
This commit is contained in:
Xuan 2015-02-20 14:02:40 -08:00
parent 187e081d5a
commit 9cbe9bcbaa
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hadoop-yarn-project/CHANGES.txt
View File

@ -276,6 +276,9 @@ Release 2.7.0 - UNRELEASED
YARN-3230. Clarify application states on the web UI. (Jian He via wangda) YARN-3230. Clarify application states on the web UI. (Jian He via wangda)
YARN-3237. AppLogAggregatorImpl fails to log error cause.
(Rushabh S Shah via xgong)
OPTIMIZATIONS OPTIMIZATIONS
YARN-2990. FairScheduler's delay-scheduling always waits for node-local and YARN-2990. FairScheduler's delay-scheduling always waits for node-local and

4
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

@ -248,7 +248,7 @@ public class AppLogAggregatorImpl implements AppLogAggregator {
} catch (IOException e1) { } catch (IOException e1) {
LOG.error("Cannot create writer for app " + this.applicationId LOG.error("Cannot create writer for app " + this.applicationId
+ ". Skip log upload this time. "); + ". Skip log upload this time. ", e1);
return; return;
} }
@ -549,7 +549,7 @@ public class AppLogAggregatorImpl implements AppLogAggregator {
writer.append(logKey, logValue); writer.append(logKey, logValue);
} catch (Exception e) { } catch (Exception e) {
LOG.error("Couldn't upload logs for " + containerId LOG.error("Couldn't upload logs for " + containerId
+ ". Skipping this container."); + ". Skipping this container.", e);
return new HashSet<Path>(); return new HashSet<Path>();
} }
this.uploadedFileMeta.addAll(logValue this.uploadedFileMeta.addAll(logValue