You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ds...@apache.org on 2014/08/21 13:42:58 UTC
git commit: AMBARI-6734 ambari-server.log size is growing rapidly and
contains lots of excess WARN records (dsen)
Repository: ambari
Updated Branches:
refs/heads/trunk 9434e18be -> 9d4164c9c
AMBARI-6734 ambari-server.log size is growing rapidly and contains lots of excess WARN records (dsen)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/9d4164c9
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/9d4164c9
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/9d4164c9
Branch: refs/heads/trunk
Commit: 9d4164c9cf24bf660a77964f91cb8249b6d1495c
Parents: 9434e18
Author: Dmytro Sen <ds...@hortonworks.com>
Authored: Thu Aug 21 14:42:59 2014 +0300
Committer: Dmytro Sen <ds...@hortonworks.com>
Committed: Thu Aug 21 14:42:59 2014 +0300
----------------------------------------------------------------------
.../ambari/server/controller/ganglia/GangliaPropertyProvider.java | 2 +-
.../apache/ambari/server/controller/jmx/JMXPropertyProvider.java | 1 -
.../java/org/apache/ambari/server/state/cluster/ClusterImpl.java | 2 +-
.../ambari/server/state/svccomphost/ServiceComponentHostImpl.java | 2 +-
4 files changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/9d4164c9/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
index 4d623c1..39a6b61 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
@@ -552,7 +552,7 @@ public abstract class GangliaPropertyProvider extends AbstractPropertyProvider {
}
} catch (IOException e) {
if (LOG.isErrorEnabled()) {
- LOG.error("Caught exception getting Ganglia metrics : spec=" + spec, e);
+ LOG.error("Caught exception getting Ganglia metrics : spec=" + spec);
}
} finally {
if (reader != null) {
http://git-wip-us.apache.org/repos/asf/ambari/blob/9d4164c9/ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java
index 4437ce9..b5a5059 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java
@@ -551,7 +551,6 @@ public class JMXPropertyProvider extends AbstractPropertyProvider {
private static String logException(Throwable throwable) {
String msg = "Caught exception getting JMX metrics : " + throwable.getLocalizedMessage();
- LOG.error(msg);
LOG.debug(msg, throwable);
return msg;
http://git-wip-us.apache.org/repos/asf/ambari/blob/9d4164c9/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java b/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
index c565060..baf58e7 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
@@ -1854,7 +1854,7 @@ public class ClusterImpl implements Cluster {
ServiceComponentHost serviceComponentHost = serviceComponent.getServiceComponentHost(event.getHostName());
serviceComponentHost.handleEvent(event);
} catch (AmbariException e) {
- LOG.error("ServiceComponentHost lookup exception ", e);
+ LOG.error("ServiceComponentHost lookup exception ", e.getMessage());
failedEvents.add(event);
} catch (InvalidStateTransitionException e) {
LOG.error("Invalid transition ", e);
http://git-wip-us.apache.org/repos/asf/ambari/blob/9d4164c9/ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java b/ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
index 3ee3078..ade5792 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
@@ -741,7 +741,7 @@ public class ServiceComponentHostImpl implements ServiceComponentHost {
saveIfPersisted();
// TODO Audit logs
} catch (InvalidStateTransitionException e) {
- LOG.error("Can't handle ServiceComponentHostEvent event at"
+ LOG.debug("Can't handle ServiceComponentHostEvent event at"
+ " current state"
+ ", serviceComponentName=" + this.getServiceComponentName()
+ ", hostName=" + this.getHostName()