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

MAPREDUCE-6418. MRApp should not shutdown LogManager during shutdown. Contributed by Chang Li

This commit is contained in:
Jason Lowe 2015-07-01 18:01:22 +00:00
parent 826715622e
commit eac1d18943
3 changed files with 29 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -527,6 +527,9 @@ Release 2.8.0 - UNRELEASED
MAPREDUCE-6420. Interrupted Exception in LocalContainerLauncher should be
logged in warn/info level (Chang Li via jlowe)
MAPREDUCE-6418. MRApp should not shutdown LogManager during shutdown
(Chang Li via jlowe)
Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES

12
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/MRAppMaster.java
View File

@ -1198,10 +1198,14 @@ public class MRAppMaster extends CompositeService {
}
}
protected void shutdownTaskLog() {
TaskLog.syncLogsShutdown(logSyncer);
}
@Override
public void stop() {
super.stop();
TaskLog.syncLogsShutdown(logSyncer);
shutdownTaskLog();
}
private boolean isRecoverySupported() throws IOException {
@ -1705,10 +1709,14 @@ public class MRAppMaster extends CompositeService {
T call(Configuration conf) throws Exception;
}
protected void shutdownLogManager() {
LogManager.shutdown();
}
@Override
protected void serviceStop() throws Exception {
super.serviceStop();
LogManager.shutdown();
shutdownLogManager();
}
public ClientService getClientService() {

15
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/MRApp.java
View File

@ -804,5 +804,20 @@ public class MRApp extends MRAppMaster {
new Text(containerToken.getService()));
return token.decodeIdentifier();
}
@Override
protected void shutdownTaskLog() {
// Avoid closing the logging system during unit tests,
// otherwise subsequent MRApp instances in the same test
// will fail to log anything.
}
@Override
protected void shutdownLogManager() {
// Avoid closing the logging system during unit tests,
// otherwise subsequent MRApp instances in the same test
// will fail to log anything.
}
}