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

(cherry picked from commit a4a30c789146c4e824fcc6ebcf10405711120eaa)

hadoop-yarn-project/CHANGES.txt

@@ -136,6 +136,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

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