Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix race accessing topic replicas map #1866

Closed
dgnorton opened this issue Mar 6, 2015 · 0 comments
Closed

Fix race accessing topic replicas map #1866

dgnorton opened this issue Mar 6, 2015 · 0 comments
Assignees

Comments

@dgnorton
Copy link
Contributor

dgnorton commented Mar 6, 2015

==================
WARNING: DATA RACE
Read by goroutine 8:
  runtime.mapiterinit()
      /home/dgnorton/src/go/src/runtime/hashmap.go:543 +0x0
  github.com/influxdb/influxdb/messaging.(*topic).encode()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/broker.go:893 +0x777
  github.com/influxdb/influxdb/messaging.(*brokerFSM).MustApply()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/broker.go:591 +0x2c5
  github.com/influxdb/influxdb/raft.func·008()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/raft/log.go:1212 +0x1760
  github.com/influxdb/influxdb/raft.(*Log).applier()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/raft/log.go:1220 +0x226

Previous write by goroutine 60:
  runtime.mapassign1()
      /home/dgnorton/src/go/src/runtime/hashmap.go:383 +0x0
  github.com/influxdb/influxdb/messaging.(*Replica).WriteTo()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/broker.go:1013 +0x96f
  github.com/influxdb/influxdb/messaging.(*Handler).stream()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/handler.go:101 +0x250
  github.com/influxdb/influxdb/messaging.(*Handler).ServeHTTP()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/handler.go:54 +0x24e
  main.(*Handler).ServeHTTP()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/handler.go:32 +0x349
  net/http.serverHandler.ServeHTTP()
      /home/dgnorton/src/go/src/net/http/server.go:1703 +0x1f6
  net/http.(*conn).serve()
      /home/dgnorton/src/go/src/net/http/server.go:1204 +0x1087

Goroutine 8 (running) created at:
  github.com/influxdb/influxdb/raft.(*Log).Open()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/raft/log.go:275 +0x705
  github.com/influxdb/influxdb/messaging.(*Broker).Open()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/messaging/broker.go:102 +0x36f
  main.openBroker()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/run.go:219 +0x10e
  main.Run()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/run.go:45 +0x447
  main.execRun()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/main.go:118 +0x8e1
  main.main()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/main.go:70 +0x4a9

Goroutine 60 (running) created at:
  net/http.(*Server).Serve()
      /home/dgnorton/src/go/src/net/http/server.go:1751 +0x3cd
  net/http.Serve()
      /home/dgnorton/src/go/src/net/http/server.go:1606 +0xdc
  main.func·003()
      /home/dgnorton/go/src/github.com/influxdb/influxdb/cmd/influxd/run.go:56 +0xb2
==================
@dgnorton dgnorton self-assigned this Mar 6, 2015
dgnorton added a commit that referenced this issue Mar 6, 2015
fix #1866: race accessing topic replicas map
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant