-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
Remove known broken code & cleanup #56
Conversation
Pull Request Test Coverage Report for Build 337
💛 - Coveralls |
27a03bc
to
a42ce1c
Compare
11da08e
to
90e72b7
Compare
Ok, so at this stage I fixed up the test runner to minimise re-connection, and do better resource isolation. And even fixed some resource leakage. |
c104a58
to
f0d59f4
Compare
rebased off master |
f0d59f4
to
f3fc36c
Compare
9472808
to
306aa9a
Compare
This PR is getting a bit bit and confused. I removed a lot of code, then added a lot of code... so it just looks like churn
I feel I should break this in at least two PRs. |
856503f
to
2b8e2e6
Compare
@Zeliboba5 Please review. |
Since pooled connections are known broken, This PR removes it, and cleans up the code, so that it will be easier to re-add it plainly.