Merge pull request #806 from cb372/datadog-remove-error-propagation

Datadog: do not propagate error to root span

Author: mpilquist

modules/datadog/src/main/scala/DDSpan.scala

@@ -15,7 +15,6 @@ import io.opentracing.propagation.{Format, TextMapAdapter}
 import io.opentracing.tag.Tags
 import natchez.TraceValue.{BooleanValue, NumberValue, StringValue}
 import _root_.datadog.trace.api.DDTags
-import _root_.datadog.trace.api.interceptor.MutableSpan
 import natchez.Span.Options
 import natchez.datadog.DDTracer.{addLink, addSpanKind}
 
@@ -101,17 +100,7 @@ final case class DDSpan[F[_]: Sync](
       DDTags.ERROR_MSG -> err.getMessage,
       DDTags.ERROR_TYPE -> err.getClass.getSimpleName,
       DDTags.ERROR_STACK -> err.getStackTrace.mkString
-    ) >> {
-      // Set error on root span
-      span match {
-        case ms: MutableSpan =>
-          Sync[F].delay {
-            val localRootSpan = ms.getLocalRootSpan
-            localRootSpan.setError(true)
-          }.void
-        case _ => Sync[F].unit
-      }
-    } >>
+    ) >>
       Sync[F].delay {
         span.log(
           (Map(