opentracing-contrib / go-stdlib

When build a request and call TraceRequest(), two spans will be recorded, a root span and a child span, so why we use two spans to trace one request?

FWIW, this is what I see in Datadog, an empty HTTP_CLIENT root span that's not necessary:

Based on the code, I don't see where a root span could have been set other than in Tracer.start(req)

Line 222 in 8a6ff1a

h.root = root

and AFAICT start only gets called once

Line 173 in 8a6ff1a

tracer.start(req)

Was Tracer suppose to be initiated with a root span somehow but got missed

Line 131 in 8a6ff1a

ht := &Tracer{tr: tr, opts: opts}

Like:

	ht := &Tracer{tr: tr, opts: opts, root: opentracing.SpanFromContext(req.Context())}

?

Why we need a root span?