Repository: hadoop
Updated Branches:
  refs/heads/yarn-2877 5f99ddbc6 -> fd6a6da2a (forced update)


YARN-4669. Fix logging statements in resource manager's Application class. 
Contributed by Sidharta Seethana.


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

Branch: refs/heads/yarn-2877
Commit: 22a2b2231d2601eb8743cc2801653e7612123f48
Parents: a6728a0
Author: Varun Vasudev <vvasu...@apache.org>
Authored: Thu Feb 4 13:48:22 2016 +0530
Committer: Varun Vasudev <vvasu...@apache.org>
Committed: Thu Feb 4 13:51:25 2016 +0530

----------------------------------------------------------------------
 hadoop-yarn-project/CHANGES.txt                           |  3 +++
 .../hadoop/yarn/server/resourcemanager/Application.java   | 10 +++++++---
 2 files changed, 10 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/22a2b223/hadoop-yarn-project/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt
index 3cf173a..e551a2a 100644
--- a/hadoop-yarn-project/CHANGES.txt
+++ b/hadoop-yarn-project/CHANGES.txt
@@ -194,6 +194,9 @@ Release 2.9.0 - UNRELEASED
     YARN-4594. container-executor fails to remove directory tree when chmod
     required (Colin Patrick McCabe via jlowe)
 
+    YARN-4669. Fix logging statements in resource manager's Application class.
+    (Seethana Sidharta via vvasudev)
+
 Release 2.8.0 - UNRELEASED
 
   INCOMPATIBLE CHANGES

http://git-wip-us.apache.org/repos/asf/hadoop/blob/22a2b223/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/Application.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/Application.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/Application.java
index b536546..17cf885 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/Application.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/Application.java
@@ -324,9 +324,13 @@ public class Application {
     Allocation allocation = resourceManager.getResourceScheduler().allocate(
         applicationAttemptId, new ArrayList<ResourceRequest>(ask),
         new ArrayList<ContainerId>(), null, null, null, null);
-    System.out.println("-=======" + applicationAttemptId);
-    System.out.println("----------" + 
resourceManager.getRMContext().getRMApps()
-        .get(applicationId).getRMAppAttempt(applicationAttemptId));
+
+    if (LOG.isInfoEnabled()) {
+      LOG.info("-=======" + applicationAttemptId + System.lineSeparator() +
+          "----------" + resourceManager.getRMContext().getRMApps()
+              .get(applicationId).getRMAppAttempt(applicationAttemptId));
+    }
+
     List<Container> containers = allocation.getContainers();
 
     // Clear state for next interaction with ResourceManager

Reply via email to