Release db connections after each callback.#24
Release db connections after each callback.#24hebnern wants to merge 1 commit intongauthier:masterfrom
Conversation
|
So, do you have to check one out each time, or does rails check it out when you make the call? I get what you were saying now in the other issue. If there are a lot of high activity frequent connections there's still gonna be a problem. I don't have a project I'm using tubesock on right now, have you run this code much on your own app? |
|
Yes, rails will check out a new connection when you do any ActiveRecord operations that need to go to the db. I ran it last night quickly. Not with heavy load, but it didn't crash at least. |
|
OK. When you said "do this for the user when we know they're done" I was thinking after close, not after all the callbacks. I get it now. Would you mind running the code a bit, then pinging me here later? We don't have a lot of tests for this sort of thing. |
|
I can submit a PR for this if people are still interested in the feature. Currently, we just wrap all code that uses ActiveRecord with |
Releases db connections after each callback is called so that a small number of db connections can be shared by a larger number of websocket connections. Proposed fix for #11