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 for unusable ORM Broker connections #126

Merged
merged 1 commit into from
Jan 8, 2016

Conversation

kdmukai
Copy link

@kdmukai kdmukai commented Dec 10, 2015

Testing on staging server has looked good. The qcluster was left idle for over 8 hours (MySQL's default wait_timeout) and then responded properly. Waited another 8+ hrs and the same cluster continued to respond properly.

References:
#124

@codecov-io
Copy link

Current coverage is 90.25%

Merging #126 into master will decrease coverage by -6.25% as of 4a068ce

@@            master    #126   diff @@
======================================
  Files           39      39       
  Stmts         2664    2669     +5
  Branches         0       0       
  Methods          0       0       
======================================
- Hit           2571    2409   -162
  Partial          0       0       
- Missed          93     260   +167

Review entire Coverage Diff as of 4a068ce

Powered by Codecov. Updated on successful CI builds.

@Koed00
Copy link
Owner

Koed00 commented Jan 7, 2016

Ok. I'm finally sort of up and running again. If this one is still relevant, I'll rerun the builds and try to merge it.

Koed00 added a commit that referenced this pull request Jan 8, 2016
Fix for stale ORM Broker connections
@Koed00 Koed00 merged commit b6dd209 into Koed00:master Jan 8, 2016
@kdmukai kdmukai deleted the close_connection branch February 6, 2016 15:23
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

Successfully merging this pull request may close these issues.

3 participants