-
-
Notifications
You must be signed in to change notification settings - Fork 410
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
Pooling #229
Pooling #229
Conversation
Pull Request Test Coverage Report for Build 983
💛 - Coveralls |
Woohoo! This is looking very good! 🎉 So close... It's probably ready for you to use it with your own project and see if it resolves the QoS issues? I wondered why the code coverage seems to have dropped so much, but looking at the report on Travis... Something weird, because basic code like doing inserts with SQLite has no coverage... 😕 |
Right, other than the nested transaction test, you need to add a changelog. Make the version 0.15.0 and have a nice short blurb about how to use this, and what it provides you. |
Thank you!, this looks great. 👍 |
i hope it works great, i assume those fixes will also be on this branch? i can then use this branch in the dev version of my project for a while to test it even more to make sure it doesn't break |
working on pooling, not done yet
Description
will give more details later on the implementation but basically just uses a pool instead of a direct connection
Motivation and Context
should significantly boost performance
How Has This Been Tested?
Checklist: