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

YARN-3641. NodeManager: stopRecoveryStore() shouldn't be skipped when exceptions happen in stopping NM's sub-services. Contributed by Junping Du 

(cherry picked from commit 711d77cc54)
This commit is contained in:
Jason Lowe 2015-05-13 21:06:47 +00:00
parent 7110499817
commit 5161751433
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -92,6 +92,9 @@ Release 2.7.1 - UNRELEASED
YARN-3537. NPE when NodeManager.serviceInit fails and stopRecoveryStore YARN-3537. NPE when NodeManager.serviceInit fails and stopRecoveryStore
invoked (Brahma Reddy Battula via jlowe) invoked (Brahma Reddy Battula via jlowe)
YARN-3641. NodeManager: stopRecoveryStore() shouldn't be skipped when
exceptions happen in stopping NM's sub-services. (Junping Du via jlowe)
Release 2.7.0 - 2015-04-20 Release 2.7.0 - 2015-04-20
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

11
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java
View File

@ -274,9 +274,14 @@ public class NodeManager extends CompositeService
if (isStopping.getAndSet(true)) { if (isStopping.getAndSet(true)) {
return; return;
} }
super.serviceStop(); try {
stopRecoveryStore(); super.serviceStop();
DefaultMetricsSystem.shutdown(); DefaultMetricsSystem.shutdown();
} finally {
// YARN-3641: NM's services stop get failed shouldn't block the
// release of NMLevelDBStore.
stopRecoveryStore();
}
} }
public String getName() { public String getName() {