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

Race in Broker.Close()/Broker.RunContinousQueryProcessing() #2324

Closed
jwilder opened this issue Apr 17, 2015 · 0 comments
Closed

Race in Broker.Close()/Broker.RunContinousQueryProcessing() #2324

jwilder opened this issue Apr 17, 2015 · 0 comments
Assignees

Comments

@jwilder
Copy link
Contributor

jwilder commented Apr 17, 2015

WARNING: DATA RACE
Read by goroutine 82:
github.com/influxdb/influxdb.(_Broker).requestContinuousQueryProcessing()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/broker.go:127 +0xbe
github.com/influxdb/influxdb.(_Broker).runContinuousQueries()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/broker.go:112 +0x473
github.com/influxdb/influxdb.(*Broker).continuousQueryLoop()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/broker.go:82 +0x76

Previous write by goroutine 470:
github.com/influxdb/influxdb.(_Broker).Close()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/broker.go:72 +0x112
github.com/influxdb/influxdb/cmd/influxd.(_Node).Close()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/run.go:72 +0x1ce
github.com/influxdb/influxdb/cmd/influxd_test.(*Cluster).Close()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:67 +0xca
github.com/influxdb/influxdb/cmd/influxd_test.Test3NodeServerFailover()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:1424 +0x579
github.com/influxdb/influxdb/cmd/influxd_test.createCombinedNodeCluster()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:134 +0x19be
github.com/influxdb/influxdb/cmd/influxd_test.Test3NodeServerFailover()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:1416 +0x38c
testing.tRunner()
/home/ubuntu/.gvm/gos/go1.4/src/testing/testing.go:447 +0x133

Goroutine 82 (running) created at:
github.com/influxdb/influxdb.(_Broker).RunContinuousQueryLoop()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/broker.go:58 +0xb6
github.com/influxdb/influxdb/cmd/influxd.(_RunCommand).Open()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/run.go:432 +0x3c0f
github.com/influxdb/influxdb/cmd/influxd_test.createCombinedNodeCluster()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:110 +0xe70
github.com/influxdb/influxdb/cmd/influxd_test.Test3NodeServerFailover()
/home/ubuntu/influxdb-build/src/github.com/influxdb/influxdb/cmd/influxd/server_integration_test.go:1416 +0x38c
testing.tRunner()
/home/ubuntu/.gvm/gos/go1.4/src/testing/testing.go:447 +0x133

Goroutine 470 (finished) created at:
testing.RunTests()
/home/ubuntu/.gvm/gos/go1.4/src/testing/testing.go:555 +0xd4e
testing.(*M).Run()
/home/ubuntu/.gvm/gos/go1.4/src/testing/testing.go:485 +0xe0
main.main()
github.com/influxdb/influxdb/cmd/influxd/_test/_testmain.go:104 +0x28c

@jwilder jwilder self-assigned this Apr 17, 2015
jwilder added a commit that referenced this issue Apr 17, 2015
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