Rename global {Set,}TraceProvider

Resolves open-telemetry#1101

api/global/internal/state.go

@@ -47,15 +47,15 @@ var (
 	delegateTraceOnce sync.Once
 )
 
-// TraceProvider is the internal implementation for global.TraceProvider.
-func TraceProvider() trace.Provider {
+// TracerProvider is the internal implementation for global.TracerProvider.
+func TracerProvider() trace.Provider {
 	return globalTracer.Load().(traceProviderHolder).tp
 }
 
-// SetTraceProvider is the internal implementation for global.SetTraceProvider.
-func SetTraceProvider(tp trace.Provider) {
+// SetTracerProvider is the internal implementation for global.SetTracerProvider.
+func SetTracerProvider(tp trace.Provider) {
 	delegateTraceOnce.Do(func() {
-		current := TraceProvider()
+		current := TracerProvider()
 		if current == tp {
 			// Setting the provider to the prior default is nonsense, panic.
 			// Panic is acceptable because we are likely still early in the

api/global/internal/trace_test.go

@@ -29,14 +29,14 @@ func TestTraceWithSDK(t *testing.T) {
 	internal.ResetForTest()
 
 	ctx := context.Background()
-	gtp := global.TraceProvider()
+	gtp := global.TracerProvider()
 	tracer1 := gtp.Tracer("pre")
 	// This is started before an SDK was registered and should be dropped.
 	_, span1 := tracer1.Start(ctx, "span1")
 
 	sr := new(tracetest.StandardSpanRecorder)
 	tp := tracetest.NewProvider(tracetest.WithSpanRecorder(sr))
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 
 	// This span was started before initialization, it is expected to be dropped.
 	span1.End()

api/global/trace.go

@@ -22,23 +22,23 @@ import (
 // Tracer creates a named tracer that implements Tracer interface.
 // If the name is an empty string then provider uses default name.
 //
-// This is short for TraceProvider().Tracer(name)
+// This is short for TracerProvider().Tracer(name)
 func Tracer(name string) trace.Tracer {
-	return TraceProvider().Tracer(name)
+	return TracerProvider().Tracer(name)
 }
 
-// TraceProvider returns the registered global trace provider.
+// TracerProvider returns the registered global trace provider.
 // If none is registered then an instance of trace.NoopProvider is returned.
 //
 // Use the trace provider to create a named tracer. E.g.
-//     tracer := global.TraceProvider().Tracer("example.com/foo")
+//     tracer := global.TracerProvider().Tracer("example.com/foo")
 // or
 //     tracer := global.Tracer("example.com/foo")
-func TraceProvider() trace.Provider {
-	return internal.TraceProvider()
+func TracerProvider() trace.Provider {
+	return internal.TracerProvider()
 }
 
-// SetTraceProvider registers `tp` as the global trace provider.
-func SetTraceProvider(tp trace.Provider) {
-	internal.SetTraceProvider(tp)
+// SetTracerProvider registers `tp` as the global trace provider.
+func SetTracerProvider(tp trace.Provider) {
+	internal.SetTracerProvider(tp)
 }

api/global/trace_test.go

@@ -32,10 +32,10 @@ func (*testTraceProvider) Tracer(_ string, _ ...trace.TracerOption) trace.Tracer
 func TestMultipleGlobalTracerProvider(t *testing.T) {
 	p1 := testTraceProvider{}
 	p2 := trace.NoopProvider{}
-	global.SetTraceProvider(&p1)
-	global.SetTraceProvider(&p2)
+	global.SetTracerProvider(&p1)
+	global.SetTracerProvider(&p2)
 
-	got := global.TraceProvider()
+	got := global.TracerProvider()
 	want := &p2
 	if got != want {
 		t.Fatalf("Provider: got %p, want %p\n", got, want)

bridge/opentracing/mix_test.go

@@ -130,7 +130,7 @@ func TestMixedAPIs(t *testing.T) {
 			t.Log(msg)
 		})
 
-		otelglobal.SetTraceProvider(otelProvider)
+		otelglobal.SetTracerProvider(otelProvider)
 		ot.SetGlobalTracer(otTracer)
 
 		tc.setup(t, mockOtelTracer)

example/namedtracer/main.go

@@ -48,7 +48,7 @@ func initTracer() {
 	if err != nil {
 		log.Panicf("failed to initialize trace provider %v\n", err)
 	}
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 }
 
 func main() {

example/otel-collector/main.go

@@ -73,7 +73,7 @@ func initProvider() (*otlp.Exporter, *push.Controller) {
 		push.WithPeriod(2*time.Second),
 	)
 
-	global.SetTraceProvider(traceProvider)
+	global.SetTracerProvider(traceProvider)
 	global.SetMeterProvider(pusher.Provider())
 	pusher.Start()
 

example/zipkin/main.go

@@ -57,7 +57,7 @@ func main() {
 
 	ctx := context.Background()
 
-	tr := global.TraceProvider().Tracer("component-main")
+	tr := global.TracerProvider().Tracer("component-main")
 	ctx, span := tr.Start(ctx, "foo")
 	<-time.After(6 * time.Millisecond)
 	bar(ctx)
@@ -69,7 +69,7 @@ func main() {
 }
 
 func bar(ctx context.Context) {
-	tr := global.TraceProvider().Tracer("component-bar")
+	tr := global.TracerProvider().Tracer("component-bar")
 	_, span := tr.Start(ctx, "bar")
 	<-time.After(6 * time.Millisecond)
 	span.End()

exporters/otlp/example_test.go

@@ -46,7 +46,7 @@ func Example_insecure() {
 		log.Fatalf("error creating trace provider: %v\n", err)
 	}
 
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 
 	tracer := global.Tracer("test-tracer")
 
@@ -87,7 +87,7 @@ func Example_withTLS() {
 		log.Fatalf("error creating trace provider: %v\n", err)
 	}
 
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 
 	tracer := global.Tracer("test-tracer")
 

exporters/stdout/example_test.go

@@ -31,7 +31,7 @@ const (
 )
 
 var (
-	tracer = global.TraceProvider().Tracer(
+	tracer = global.TracerProvider().Tracer(
 		instrumentationName,
 		trace.WithInstrumentationVersion(instrumentationVersion),
 	)

exporters/stdout/exporter.go

@@ -92,7 +92,7 @@ func InstallNewPipeline(exportOpts []Option, pushOpts []push.Option) (*push.Cont
 	if err != nil {
 		return controller, err
 	}
-	global.SetTraceProvider(traceProvider)
+	global.SetTracerProvider(traceProvider)
 	global.SetMeterProvider(controller.Provider())
 	return controller, err
 }

exporters/trace/jaeger/jaeger.go

@@ -178,7 +178,7 @@ func InstallNewPipeline(endpointOption EndpointOption, opts ...Option) (func(),
 		return nil, err
 	}
 
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 	return flushFn, nil
 }
 

exporters/trace/jaeger/jaeger_test.go

@@ -82,9 +82,9 @@ func TestInstallNewPipeline(t *testing.T) {
 			defer fn()
 
 			assert.NoError(t, err)
-			assert.IsType(t, tc.expectedProvider, global.TraceProvider())
+			assert.IsType(t, tc.expectedProvider, global.TracerProvider())
 
-			global.SetTraceProvider(nil)
+			global.SetTracerProvider(nil)
 		})
 	}
 }
@@ -145,7 +145,7 @@ func TestNewExportPipeline(t *testing.T) {
 			defer fn()
 
 			assert.NoError(t, err)
-			assert.NotEqual(t, tp, global.TraceProvider())
+			assert.NotEqual(t, tp, global.TracerProvider())
 			assert.IsType(t, tc.expectedProviderType, tp)
 
 			if tc.testSpanSampling {
@@ -345,7 +345,7 @@ func TestExporter_ExportSpan(t *testing.T) {
 
 	assert.NoError(t, err)
 
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 	_, span := global.Tracer("test-tracer").Start(context.Background(), "test-span")
 	span.End()
 

exporters/trace/zipkin/zipkin.go

@@ -133,7 +133,7 @@ func InstallNewPipeline(collectorURL, serviceName string, opts ...Option) error
 		return err
 	}
 
-	global.SetTraceProvider(tp)
+	global.SetTracerProvider(tp)
 	return nil
 }
 

exporters/trace/zipkin/zipkin_test.go

@@ -48,7 +48,7 @@ func TestInstallNewPipeline(t *testing.T) {
 		serviceName,
 	)
 	assert.NoError(t, err)
-	assert.IsType(t, &sdktrace.Provider{}, global.TraceProvider())
+	assert.IsType(t, &sdktrace.Provider{}, global.TracerProvider())
 }
 
 func TestNewExportPipeline(t *testing.T) {
@@ -90,7 +90,7 @@ func TestNewExportPipeline(t *testing.T) {
 				tc.options...,
 			)
 			assert.NoError(t, err)
-			assert.NotEqual(t, tp, global.TraceProvider())
+			assert.NotEqual(t, tp, global.TracerProvider())
 
 			if tc.testSpanSampling {
 				_, span := tp.Tracer("zipkin test").Start(context.Background(), tc.name)