Skip to content

Commit

Permalink
remove locking from jaeger receiver start and stop processes (open-te…
Browse files Browse the repository at this point in the history
…lemetry#3070)

* remove locking from jaeger receiver start and stop processes

* remove mutex field
  • Loading branch information
bhautikpip committed May 5, 2021
1 parent 0301927 commit 18ed27f
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions receiver/jaegerreceiver/trace_receiver.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,9 +76,6 @@ type configuration struct {
// Receiver type is used to receive spans that were originally intended to be sent to Jaeger.
// This receiver is basically a Jaeger collector.
type jReceiver struct {
// mu protects the fields of this type
mu sync.Mutex

nextConsumer consumer.Traces
instanceName string

Expand Down Expand Up @@ -182,9 +179,6 @@ func (jr *jReceiver) collectorHTTPEnabled() bool {
}

func (jr *jReceiver) Start(_ context.Context, host component.Host) error {
jr.mu.Lock()
defer jr.mu.Unlock()

if err := jr.startAgent(host); err != nil {
return err
}
Expand All @@ -197,8 +191,6 @@ func (jr *jReceiver) Start(_ context.Context, host component.Host) error {
}

func (jr *jReceiver) Shutdown(ctx context.Context) error {
jr.mu.Lock()
defer jr.mu.Unlock()
var errs []error

if jr.agentServer != nil {
Expand Down

0 comments on commit 18ed27f

Please sign in to comment.