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 <g...@brownuk.com> Authored: Thu Apr 13 11:51:18 2017 +0100 Committer: Claus Ibsen <davscl...@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; }
