denisenkom / pytds

Python DBAPI driver for MSSQL using pure Python TDS (Tabular Data Stream) protocol implementation
MIT License
192 stars 53 forks source link

Switch to namedlist for python3 compatibility #72

Closed tpow closed 7 years ago

tpow commented 7 years ago

According to the creator of recordtype it is obsolete. See https://bitbucket.org/ericvsmith/recordtype

It also doesn't work properly with Python 3.x. It looks like namedlist is a compatible replacement that IS python 3 compatible.

denisenkom commented 7 years ago

Should this be a separate strategy?

tpow commented 7 years ago

I wouldn't think it would need to be a separate strategy. It's entirely compatible. For the point of view of the calling code, it would work exactly the same. It simply would use a different module behind the scenes.

I tried to make it so that existing code (2.x) that is using recordtype would just keep working, but if you needed python 3 compatibility or wanted to switch to namedlist it could use it.