1
0
mirror of https://github.com/open-telemetry/opentelemetry-go.git synced 2025-06-04 23:07:40 +02:00

Misc fixes from PR 381 (#443)

* Remove Vendor constants from tracing plugins

Unused. And confusing, since "ot" may mean "opentracing" as well.

* Simplify current span key declaration

No need for a block.

* Fix typo

Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
This commit is contained in:
Krzesimir Nowak 2020-01-30 19:25:53 +01:00 committed by GitHub
parent 6b4acf47b8
commit 405a92ab3e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 2 additions and 13 deletions

View File

@ -20,9 +20,7 @@ import (
type currentSpanKeyType struct{}
var (
currentSpanKey = &currentSpanKeyType{}
)
var currentSpanKey = &currentSpanKeyType{}
func ContextWithSpan(ctx context.Context, span Span) context.Context {
return context.WithValue(ctx, currentSpanKey, span)

View File

@ -24,11 +24,6 @@ import (
"go.opentelemetry.io/otel/api/trace"
)
const (
// Vendor is the integration provider
Vendor = "ot"
)
var (
propagator = trace.TraceContext{}
)

View File

@ -23,10 +23,6 @@ import (
"go.opentelemetry.io/otel/api/trace"
)
const (
Vendor = "ot"
)
var (
HostKey = key.New("http.host")
URLKey = key.New("http.url")

View File

@ -76,7 +76,7 @@ func ExampleNewHandler() {
func(w http.ResponseWriter, r *http.Request) {
ctx := r.Context()
var name string
// Wrap another function in it's own span
// Wrap another function in its own span
if err := trace.SpanFromContext(ctx).Tracer().WithSpan(ctx, "figureOutName",
func(ctx context.Context) error {
var err error