[camel] branch master updated: CAMEL-11878: Reduce logging noise and polish.

[davsclaus]

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 4af997d  CAMEL-11878: Reduce logging noise and polish.
4af997d is described below

commit 4af997d2707a084f9c8a36f7dc9b1a2fea5a9a10
Author: Claus Ibsen <claus.ib...@gmail.com>
AuthorDate: Thu Feb 1 16:09:01 2018 +0100

    CAMEL-11878: Reduce logging noise and polish.
---
 .../camel/blueprint/BlueprintCamelContext.java      | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git 
a/components/camel-blueprint/src/main/java/org/apache/camel/blueprint/BlueprintCamelContext.java
 
b/components/camel-blueprint/src/main/java/org/apache/camel/blueprint/BlueprintCamelContext.java
index 2282ce4..fe3e3ad 100644
--- 
a/components/camel-blueprint/src/main/java/org/apache/camel/blueprint/BlueprintCamelContext.java
+++ 
b/components/camel-blueprint/src/main/java/org/apache/camel/blueprint/BlueprintCamelContext.java
@@ -115,13 +115,13 @@ public class BlueprintCamelContext extends 
DefaultCamelContext implements Servic
         try {
             bundleContext.removeServiceListener(this);
         } catch (Exception e) {
-            LOG.warn("Error removing ServiceListener " + this + ". This 
exception is ignored.", e);
+            LOG.warn("Error removing ServiceListener: " + this + ". This 
exception is ignored.", e);
         }
         if (registration != null) {
             try {
                 registration.unregister();
             } catch (Exception e) {
-                LOG.warn("Error unregistering service registration " + 
registration + ". This exception is ignored.", e);
+                LOG.warn("Error unregistering service registration: " + 
registration + ". This exception is ignored.", e);
             }
             registration = null;
         }
@@ -167,33 +167,31 @@ public class BlueprintCamelContext extends 
DefaultCamelContext implements Servic
                 break;
             }
 
-            LOG.debug("Received BlueprintEvent[ replay={} type={} bundle={}] 
%s", event.isReplay(), eventTypeString, event.getBundle().getSymbolicName(), 
event.toString());
+            LOG.debug("Received BlueprintEvent[replay={} type={} bundle={}] 
%s", event.isReplay(), eventTypeString, event.getBundle().getSymbolicName(), 
event.toString());
         }
 
         if (!event.isReplay() && 
this.getBundleContext().getBundle().getBundleId() == 
event.getBundle().getBundleId()) {
             if (event.getType() == BlueprintEvent.CREATED) {
                 try {
-                    LOG.info("Attempting to start Camel Context {}", 
this.getName());
+                    LOG.info("Attempting to start CamelContext: {}", 
this.getName());
                     this.maybeStart();
                 } catch (Exception startEx) {
-                    LOG.error("Error occurred during starting Camel Context  " 
+ this.getName(), startEx);
+                    LOG.error("Error occurred during starting CamelContext: " 
+ this.getName(), startEx);
                 }
             } else if (event.getType() == BlueprintEvent.DESTROYING) {
                 try {
-                    LOG.info("Stopping Camel Context {}", this.getName());
+                    LOG.info("Stopping CamelContext: {}", this.getName());
                     this.stop();
                 } catch (Exception stopEx) {
-                    LOG.error("Error occurred during stopping Camel Context " 
+ this.getName(), stopEx);
+                    LOG.error("Error occurred during stopping CamelContext: " 
+ this.getName(), stopEx);
                 }
-
             }
         }
-
     }
 
     @Override
     public void serviceChanged(ServiceEvent event) {
-        if (LOG.isDebugEnabled()) {
+        if (LOG.isTraceEnabled()) {
             String eventTypeString;
 
             switch (event.getType()) {
@@ -214,7 +212,8 @@ public class BlueprintCamelContext extends 
DefaultCamelContext implements Servic
                 break;
             }
 
-            LOG.debug("Service {} changed to {}", event.toString(), 
eventTypeString);
+            // use trace logging as this is very noisy
+            LOG.trace("Service: {} changed to: {}", event.toString(), 
eventTypeString);
         }
     }
 

-- 
To stop receiving notification emails like this one, please contact
davscl...@apache.org.