YARN-3604. Fixed ZKRMStateStore#removeApplication to also disable watch. Contributed Zhihai Xu
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/d0b73287 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/d0b73287 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/d0b73287 Branch: refs/heads/HDFS-7240 Commit: d0b73287c26983f32b0efe01862150e9fd28086f Parents: 088156d Author: Jian He <[email protected]> Authored: Fri May 8 15:10:38 2015 -0700 Committer: Jian He <[email protected]> Committed: Fri May 8 15:12:33 2015 -0700 ---------------------------------------------------------------------- hadoop-yarn-project/CHANGES.txt | 3 +++ .../yarn/server/resourcemanager/recovery/ZKRMStateStore.java | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/d0b73287/hadoop-yarn-project/CHANGES.txt ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt index 2183d3f..6d547c1 100644 --- a/hadoop-yarn-project/CHANGES.txt +++ b/hadoop-yarn-project/CHANGES.txt @@ -368,6 +368,9 @@ Release 2.8.0 - UNRELEASED YARN-3600. AM container link is broken (Naganarasimha G R via tgraves) + YARN-3604. Fixed ZKRMStateStore#removeApplication to also disable watch. + (zhihai xu via jianhe) + Release 2.7.1 - UNRELEASED INCOMPATIBLE CHANGES http://git-wip-us.apache.org/repos/asf/hadoop/blob/d0b73287/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/ZKRMStateStore.java ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/ZKRMStateStore.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/ZKRMStateStore.java index 364c970..15ac971 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/ZKRMStateStore.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/ZKRMStateStore.java @@ -827,8 +827,8 @@ public class ZKRMStateStore extends RMStateStore { public synchronized void removeApplication(ApplicationId removeAppId) throws Exception { String appIdRemovePath = getNodePath(rmAppRoot, removeAppId.toString()); - if (existsWithRetries(appIdRemovePath, true) != null) { - deleteWithRetries(appIdRemovePath, true); + if (existsWithRetries(appIdRemovePath, false) != null) { + deleteWithRetries(appIdRemovePath, false); } }
