Repository: hadoop
Updated Branches:
  refs/heads/YARN-1011 c6870a1fd -> 36ec27ed3


YARN-8391. Investigate AllocationFileLoaderService.reloadListener locking 
issue. Contributed by Szilard Nemeth.

(cherry picked from commit 9a9e969570f23b627f9571819f388916d8fd7ec9)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/36ec27ed
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/36ec27ed
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/36ec27ed

Branch: refs/heads/YARN-1011
Commit: 36ec27ed32a07bd6be6e0427b715435c273e0c94
Parents: c6870a1
Author: Miklos Szegedi <miklos.szeg...@cloudera.com>
Authored: Wed Jun 20 10:29:12 2018 -0700
Committer: Haibo Chen <haiboc...@apache.org>
Committed: Mon Jul 30 15:20:26 2018 -0700

----------------------------------------------------------------------
 .../scheduler/fair/AllocationFileLoaderService.java              | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/36ec27ed/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.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/scheduler/fair/AllocationFileLoaderService.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java
index e541ab7..c53fe8f 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java
@@ -114,7 +114,9 @@ public class AllocationFileLoaderService extends 
AbstractService {
       reloadThread = new Thread(() -> {
         while (running) {
           try {
-            reloadListener.onCheck();
+            synchronized (this) {
+              reloadListener.onCheck();
+            }
             long time = clock.getTime();
             long lastModified =
                 fs.getFileStatus(allocFile).getModificationTime();


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org

Reply via email to