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/branch-alerts-dev Commit: 9d4164c9cf24bf660a77964f91cb8249b6d1495c Parents: 9434e18 Author: Dmytro Sen <[email protected]> Authored: Thu Aug 21 14:42:59 2014 +0300 Committer: Dmytro Sen <[email protected]> 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()
