You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by sm...@apache.org on 2014/01/07 22:59:11 UTC

git commit: AMBARI-4235. Remove unnecessary error logs and improve message

Updated Branches:
  refs/heads/trunk d507500e4 -> a0c5805e2


AMBARI-4235. Remove unnecessary error logs and improve message


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

Branch: refs/heads/trunk
Commit: a0c5805e299ef22643f4791ac4a019af24cf7ba3
Parents: d507500
Author: Sumit Mohanty <sm...@hortonworks.com>
Authored: Tue Jan 7 13:27:53 2014 -0800
Committer: Sumit Mohanty <sm...@hortonworks.com>
Committed: Tue Jan 7 13:27:53 2014 -0800

----------------------------------------------------------------------
 .../apache/ambari/server/actionmanager/ActionScheduler.java  | 4 ++--
 .../server/controller/internal/AbstractProviderModule.java   | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/a0c5805e/ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java b/ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java
index 19e4b79..02c94b8 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java
@@ -350,8 +350,8 @@ class ActionScheduler implements Runnable {
               svcCompHost.handleEvent(timeoutEvent);
               LOG.warn("Operation timed out. Role: " + roleStr + ", host: " + host);
             } catch (ServiceComponentNotFoundException scnex) {
-              LOG.debug("Not a service component, assuming its an action. Details: "
-                  + scnex.getMessage());
+              LOG.debug(roleStr + " associated with service " + c.getServiceName() +
+                  " is not a service component, assuming it's an action.");
             } catch (InvalidStateTransitionException e) {
               LOG.info("Transition failed for host: " + host + ", role: "
                   + roleStr, e);

http://git-wip-us.apache.org/repos/asf/ambari/blob/a0c5805e/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
index e5e10ef..fb14a3b 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
@@ -664,17 +664,17 @@ public abstract class AbstractProviderModule implements ProviderModule, Resource
       }
 
     } catch (Exception e) {
-      LOG.error("Exception while detecting JMX protocol for clusterName = " + clusterName +
+      LOG.info("Exception while detecting JMX protocol for clusterName = " + clusterName +
           ", componentName = " + componentName,  e);
-      LOG.error("Defaulting JMX to HTTP protocol for  for clusterName = " + clusterName +
+      LOG.info("Defaulting JMX to HTTP protocol for  for clusterName = " + clusterName +
           ", componentName = " + componentName +
           componentName);
       jmxProtocolString = "http";
     }
     if (jmxProtocolString == null) {
-      LOG.error("Detected JMX protocol is null for clusterName = " + clusterName +
+      LOG.debug("Detected JMX protocol is null for clusterName = " + clusterName +
           ", componentName = " + componentName);
-      LOG.error("Defaulting JMX to HTTP protocol for  for clusterName = " + clusterName +
+      LOG.debug("Defaulting JMX to HTTP protocol for  for clusterName = " + clusterName +
           ", componentName = " + componentName +
           componentName);
       jmxProtocolString = "http";