Use lazy connect and let server disconnect on inactivity

[mysql-otp]

Use lazy connect and let server disconnect on inactivity

[juhlig]

Sounds good to me :) Setting the connect strategy to lazy allows workers to (re-) start more quickly and reliably, and works around poolboy's annoying deficiency to crash an entire pool if one worker fails to (re-) start, too.

One ugliness that remains is that part of it (the idle timeout) depends on the server, ie there is no way for the client to decide that it wants to be extra nice and close the connection when it has not been used for a while (maybe even to just close and revert to the not-connected state of lazy mode).

[zuiderkwast]

@getong, @juhlig, other users, what's your opinion on this?

[zuiderkwast]

and works around poolboy's annoying deficiency to crash an entire pool if one worker fails to (re-) start, too

Yes, I didn't even think about this. It's an extra benefit.

maybe even to just close and revert to the not-connected state of lazy mode

Hm... Then we lose the connection state, but as a non-default option with an explicit warning, perhaps yes. Anyway, with just the server disconnecting after 8 hours idle, the application will come to some balance of how many connections it needs within 24 hours.

[juhlig]

and works around poolboy's annoying deficiency to crash an entire pool if one worker fails to (re-) start, too

Yes, I didn't even think about this. It's an extra benefit.

Oh, plus it has the benefit that workers will always start quickly. With poolboy, worker starts happen synchronously, blocking all other pool operations, so quick worker starts are good :)

maybe even to just close and revert to the not-connected state of lazy mode

Hm... Then we lose the connection state, but as a non-default option with an explicit warning, perhaps yes.

Just a thought, I think not really worth it.

Anyway, with just the server disconnecting after 8 hours idle, the application will come to some balance of how many connections it needs within 24 hours.

I guess so :) In any case, I think that this would be a good improvement, so I'm in favor of it :)