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

database/sql: goroutine leak in connectionResetter #22699

Closed
kardianos opened this issue Nov 13, 2017 · 1 comment
Closed

database/sql: goroutine leak in connectionResetter #22699

kardianos opened this issue Nov 13, 2017 · 1 comment
Assignees
Milestone

Comments

@kardianos
Copy link
Contributor

@kardianos kardianos commented Nov 13, 2017

@mattn sqlite driver included a stress test that attempts to open and close 10000 database pools. This exposed a goroutine leak in the connectinoResetter.
mattn/go-sqlite3#482

The fix is easy and this was introduced in the go1.10 cycle. CL coming soon.

@kardianos kardianos added this to the Go1.10 milestone Nov 13, 2017
@kardianos kardianos self-assigned this Nov 13, 2017
@gopherbot
Copy link

@gopherbot gopherbot commented Nov 13, 2017

Change https://golang.org/cl/77390 mentions this issue: database/sql: do not leak the connectionResetter goroutine

@gopherbot gopherbot closed this in f7df55d Nov 14, 2017
@golang golang locked and limited conversation to collaborators Nov 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.