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.

Sign up for GitHub

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

Jump to bottom

cmd/bosun: add missing redis connection close #1755

Merged

kylebrandt merged 1 commit into master from redisFix

May 27, 2016

Member

kylebrandt commented May 25, 2016

#1753 maybe


          cmd/bosun: add missing redis connection close

911e14b

Contributor

captncraig commented May 26, 2016

Yes, this is definitely a problem. The fix is good. LGTM. I wonder if we can monitor our connection pool in any way to detect this in the future.

kylebrandt merged commit e5a53e8 into master

Contributor

Dieterbe commented May 27, 2016

<3

kylebrandt deleted the redisFix branch

June 9, 2016 19:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet