Repository: hadoop
Updated Branches:
  refs/heads/branch-2 49ed955c9 -> ac0ddc4fe


YARN-4298. Fix findbugs warnings in hadoop-yarn-common. Contributed by Sunil G.

(cherry picked from commit 8c0133f3d4c85733c1e0389d1ee39f1eab58c0b9)


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

Branch: refs/heads/branch-2
Commit: ac0ddc4fec907d19a95c5e5305bb27a82d0fa085
Parents: 49ed955
Author: Akira Ajisaka <[email protected]>
Authored: Tue Nov 24 18:40:33 2015 +0900
Committer: Akira Ajisaka <[email protected]>
Committed: Tue Nov 24 18:40:56 2015 +0900

----------------------------------------------------------------------
 hadoop-yarn-project/CHANGES.txt                                  | 3 +++
 .../yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/ac0ddc4f/hadoop-yarn-project/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt
index e17883d..d2ce060 100644
--- a/hadoop-yarn-project/CHANGES.txt
+++ b/hadoop-yarn-project/CHANGES.txt
@@ -1009,6 +1009,9 @@ Release 2.8.0 - UNRELEASED
 
     YARN-4367. SLS webapp doesn't load. (kasha)
 
+    YARN-4298. Fix findbugs warnings in hadoop-yarn-common.
+    (Sunil G via aajisaka)
+
 Release 2.7.3 - UNRELEASED
 
   INCOMPATIBLE CHANGES

http://git-wip-us.apache.org/repos/asf/hadoop/blob/ac0ddc4f/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java
index bd460f6..da87465 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/impl/pb/AllocateResponsePBImpl.java
@@ -386,7 +386,7 @@ public class AllocateResponsePBImpl extends 
AllocateResponse {
   }
 
   @Override
-  public Priority getApplicationPriority() {
+  public synchronized Priority getApplicationPriority() {
     AllocateResponseProtoOrBuilder p = viaProto ? proto : builder;
     if (this.appPriority != null) {
       return this.appPriority;
@@ -399,7 +399,7 @@ public class AllocateResponsePBImpl extends 
AllocateResponse {
   }
 
   @Override
-  public void setApplicationPriority(Priority priority) {
+  public synchronized void setApplicationPriority(Priority priority) {
     maybeInitBuilder();
     if (priority == null)
       builder.clearApplicationPriority();

Reply via email to