camel-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From davscl...@apache.org
Subject [1/2] camel git commit: OpenTracing: Change error to warn based on review comment
Date Thu, 13 Apr 2017 11:02:57 GMT
Repository: camel
Updated Branches:
  refs/heads/master e269e10d4 -> e1d89ddbf


OpenTracing: Change error to warn based on review comment


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

Branch: refs/heads/master
Commit: e1d89ddbf6028369fd3117ff648fc6f16a1786fe
Parents: b5bdecd
Author: Gary Brown <gary@brownuk.com>
Authored: Thu Apr 13 11:51:18 2017 +0100
Committer: Claus Ibsen <davsclaus@apache.org>
Committed: Thu Apr 13 13:02:50 2017 +0200

----------------------------------------------------------------------
 .../org/apache/camel/opentracing/OpenTracingTracer.java | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/e1d89ddb/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/OpenTracingTracer.java
----------------------------------------------------------------------
diff --git a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/OpenTracingTracer.java
b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/OpenTracingTracer.java
index f3d15d4..3ae9fc1 100644
--- a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/OpenTracingTracer.java
+++ b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/OpenTracingTracer.java
@@ -242,7 +242,8 @@ public class OpenTracingTracer extends ServiceSupport implements RoutePolicyFact
                     managedSpan.deactivate();
                 }
             } catch (Throwable t) {
-                LOG.error("OpenTracing: Failed to capture tracing data", t);
+                // This exception is ignored
+                LOG.warn("OpenTracing: Failed to capture tracing data", t);
             }
         }
 
@@ -283,7 +284,8 @@ public class OpenTracingTracer extends ServiceSupport implements RoutePolicyFact
                     LOG.trace("OpenTracing: start server span=" + span);
                 }
             } catch (Throwable t) {
-                LOG.error("OpenTracing: Failed to capture tracing data", t);
+                // This exception is ignored
+                LOG.warn("OpenTracing: Failed to capture tracing data", t);
             }
         }
 
@@ -303,7 +305,8 @@ public class OpenTracingTracer extends ServiceSupport implements RoutePolicyFact
                     LOG.warn("OpenTracing: could not find managed span for exchange=" + exchange);
                 }
             } catch (Throwable t) {
-                LOG.error("OpenTracing: Failed to capture tracing data", t);
+                // This exception is ignored
+                LOG.warn("OpenTracing: Failed to capture tracing data", t);
             }
         }
     }
@@ -327,7 +330,8 @@ public class OpenTracingTracer extends ServiceSupport implements RoutePolicyFact
                     span.log(fields);
                 }
             } catch (Throwable t) {
-                LOG.error("OpenTracing: Failed to capture tracing data", t);
+                // This exception is ignored
+                LOG.warn("OpenTracing: Failed to capture tracing data", t);
             }
             return message;
         }


Mime
View raw message