Remove terminated connections from the pool. #51

Merged
merged 1 commit into from Dec 27, 2013

Projects

None yet

2 participants

@sontek
sontek commented Dec 16, 2013

In pymssql, if you terminate a long running query manually
it will give you a connection reset by peer message, but this
connection remains in the pool and will be re-used.

@sontek sontek Remove terminated connections from the pool.
In pymssql, if you terminate a long running query manually
it will give you a connection reset by peer message, but this
connection remains in the pool and will be re-used.
3147ca3
@sontek sontek commented on the diff Dec 16, 2013
test/dialect/mssql/test_engine.py
'message 20003',
- "Error 10054",
- "Not connected to any MS SQL server",
- "Connection is closed"
+ 'Error 10054',
+ 'Not connected to any MS SQL server',
+ 'Connection is closed'
@sontek
sontek Dec 16, 2013

The quoting was inconsistent in the file, so I cleaned it up, but can remove that if you don't like it.

@zzzeek
Owner
zzzeek commented Dec 16, 2013

ok, got a few pullreqs I need to get to, should be fine

@zzzeek zzzeek merged commit 69876ad into zzzeek:master Dec 27, 2013
@zzzeek
Owner
zzzeek commented Dec 27, 2013

changelog: c837dc4 cherry-pick in 0.8 4211207 351d0b6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment