OK that issue is fixed in master and 0.8. thanks for the report!
On Jul 4, 2013, at 12:44 PM, Navid Sheikhol Eslami <[email protected]> wrote: > > On Thursday, July 4, 2013 7:00:18 AM UTC-7, Michael Bayer wrote: > I'll have to work up a more self contained test using mocks (the approach can > be seen here: > https://github.com/zzzeek/sqlalchemy/blob/master/test/engine/test_reconnect.py#L21) > so that this can be tested completely - if overflow is counting incorrectly, > the implication is that the pool will eventually not be able to deliver > connections past the pool size, is that right? I'd like to reproduce that. > > Yes, QueuePool will eventually hit a TimeoutError (which is what made me > notice the discrepancy). > > -- > You received this message because you are subscribed to the Google Groups > "sqlalchemy" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/sqlalchemy. > For more options, visit https://groups.google.com/groups/opt_out. > > -- You received this message because you are subscribed to the Google Groups "sqlalchemy" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sqlalchemy. For more options, visit https://groups.google.com/groups/opt_out.
