jaegertracing / jaeger-client-go

🛑 This library is DEPRECATED!

Home Page:https://jaegertracing.io/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

sampler logger is NullLogger

tttoad opened this issue · comments

commented

Sampler initialization without logger injection.

func (sc *SamplerConfig) NewSampler(
	serviceName string,
	metrics *jaeger.Metrics,
) 

should be change to

func (sc *SamplerConfig) NewSampler(
	serviceName string,
	metrics *jaeger.Metrics,
	logger jaeger.Logger
) 

like

func (rc *ReporterConfig) NewReporter(
	serviceName string,
	metrics *jaeger.Metrics,
	logger jaeger.Logger,
) (jaeger.Reporter, error) {