djhenderson / pymssql

Automatically exported from code.google.com/p/pymssql
GNU Lesser General Public License v2.1
0 stars 0 forks source link

time type is not supported #103

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
pymssql supports datetime and date types, but not time for some reason.

Attached is a patch for pymssql-2.0.0b1-dev-20111019 that adds time type 
support.

Original issue reported on code.google.com by maxim.ye...@gmail.com on 15 Nov 2012 at 1:43

Attachments:

GoogleCodeExporter commented 9 years ago
This was an incomplete patch though. It only allowed inserting date's into the 
database, but not reading them back.

Original comment by maxim.ye...@gmail.com on 26 Nov 2012 at 1:44

GoogleCodeExporter commented 9 years ago
The reason we don't support time types is because SQL Server didn't support 
them until 2008.  And, I don't think FreeTDS supports the new types yet.

I sent an email to the FreeTDS list to see what the status of support for this 
is in FreeTDS.

Until FreeTDS supports this, I don't want to support it.  I don't want to be 
making assumptions in pymssql about how to save time to datetime in sql server 
and back.  Let the application developer or DB framework handle that.

Original comment by rsyr...@gmail.com on 29 Apr 2013 at 6:16

GoogleCodeExporter commented 9 years ago
Hi Randy,

Did FreeTDS ever respond? Do you think this is something we should add support 
for or should this be closed as "won't fix"?

Original comment by msabr...@gmail.com on 15 Aug 2013 at 8:32

GoogleCodeExporter commented 9 years ago
Sent to FreeTDS list:

Hi Gents,

Can you give me a status update on support for the Time type?  After the 
exchange below, the email thread takes a pretty technical turn.  From that 
discussion, it sounds like the Time type is not yet supported in dblib?

Still trying to figure out what to do with this issue:

http://code.google.com/p/pymssql/issues/detail?id=103

Thanks.

Original comment by randy.sy...@lev12.com on 16 Aug 2013 at 2:47