svn commit: r1466076 - in /incubator/ambari/trunk: ./ ambari-server/src/main/java/org/apache/ambari/server/api/query/ ambari-server/src/main/java/org/apache/ambari/server/api/services/ ambari-server/src/main/java/org/apache/ambari/server/controller/int...

[tb...@apache.org]

Author: tbeerbower
Date: Tue Apr  9 15:16:42 2013
New Revision: 1466076

URL: http://svn.apache.org/r1466076
Log:
AMBARI-1845 - Server log is being flooded with log messages

Modified:
    incubator/ambari/trunk/CHANGES.txt
    incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java
    incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java
    incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java

Modified: incubator/ambari/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1466076&r1=1466075&r2=1466076&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Tue Apr  9 15:16:42 2013
@@ -230,6 +230,9 @@ Trunk (unreleased changes):
  AMBARI-1844. Need ability to update multiple hosts in 1 PUT call.
  (tbeerbower)
 
+ AMBARI-1845. Server log is being flooded with log messages.
+ (tbeerbower)
+
  AMBARI-1825. Minor label change for the buttons to start/stop all services
  in the side nav. (yusaku)
 

Modified: incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java?rev=1466076&r1=1466075&r2=1466076&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java (original)
+++ incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java Tue Apr  9 15:16:42 2013
@@ -133,9 +133,9 @@ public class QueryImpl implements Query 
       m_mapSubResources.putAll(m_resource.getSubResources());
     }
 
-    if (LOG.isInfoEnabled()) {
+    if (LOG.isDebugEnabled()) {
       //todo: include predicate info.  Need to implement toString for all predicates.
-      LOG.info("Executing resource query: " + m_resource.getIds());
+      LOG.debug("Executing resource query: " + m_resource.getIds());
     }
 
     Predicate predicate = createPredicate(m_resource);

Modified: incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java?rev=1466076&r1=1466075&r2=1466076&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java (original)
+++ incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java Tue Apr  9 15:16:42 2013
@@ -90,7 +90,9 @@ public abstract class BaseRequest implem
 
   @Override
   public Result process() {
-    LOG.info("Handling API Request: '" + getURI() + "'");
+    if (LOG.isDebugEnabled()) {
+      LOG.debug("Handling API Request: '" + getURI() + "'");
+    }
 
     Result result;
     try {

Modified: incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java?rev=1466076&r1=1466075&r2=1466076&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java (original)
+++ incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java Tue Apr  9 15:16:42 2013
@@ -89,10 +89,11 @@ public class ClusterControllerImpl imple
     if (provider == null) {
       resources = Collections.emptySet();
     } else {
-      LOG.info("Using resource provider "
+      if (LOG.isDebugEnabled()) {
+        LOG.debug("Using resource provider "
           + provider.getClass().getName()
           + " for request type " + type.toString());
-
+      }
       checkProperties(type, request, predicate);
 
       resources = provider.getResources(request, predicate);