Opened 18 years ago

Closed 16 years ago

#1429 closed enhancement (invalid)

Yet another patch for SQL server adodbapi

Reported by: jdunck@… Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

From Matt Good, this patch works for him on top of 0.91.

Needs to be merged in against 0.91, have dj tests pass, then translated to magic-removal and pass those.

Attachments (1)

adodbapi-from-0.91.patch (19.5 KB ) - added by anonymous 18 years ago.

Download all attachments as: .zip

Change History (8)

by anonymous, 18 years ago

Attachment: adodbapi-from-0.91.patch added

comment:1 by Matt Good <django@…>, 18 years ago

Thanks for uploading that patch. I should note that in the ADO backend I made the connection a threading.local as an attempt to work around some threading problems I had, but that should probably be removed.

Also, the patch included a change I made in django.core.meta to allow nullable many-to-one relationships, which I've heard will no longer be necessary on the magic-removal branch.

The "mssql" backend in the patch is based on my "ado_mssql" backend and differs from the implementation in #1258 by parsing the LIMIT and OFFSET from the query instead of special-casing it in django.core.meta. There is a special-case for enabling IDENTITY_INSERT (though I realized it's not turned back off as done in #1258, which may be a good idea).

The pymssql module used in the "mssql" backend is currently GPL licensed, but I've contacted the authors and they are willing to relicense it under the LGPL. A new 0.8 release is supposed to be out shortly that will include the new license.

comment:2 by Adrian Holovaty, 18 years ago

#1258 was another attempt.

comment:3 by Matt Good, 18 years ago

This news is a little late, but the pymssql authors have changed the license to LGPL: http://sourceforge.net/forum/forum.php?forum_id=566339

comment:4 by Gary Wilson <gary.wilson@…>, 18 years ago

milestone: Version 0.92Version 1.0

0.92 is long gone.

comment:5 by (none), 17 years ago

milestone: Version 1.0

Milestone Version 1.0 deleted

comment:6 by Gary Wilson <gary.wilson@…>, 17 years ago

Has patch: set
Needs documentation: set
Patch needs improvement: set
Triage Stage: UnreviewedAccepted
Version: magic-removalSVN

Patch is out of date. magic-removal was already merged.

comment:7 by Malcolm Tredinnick, 16 years ago

Resolution: invalid
Status: newclosed

As per #2358 and #3702, we're asking adosql development to be done out of tree for now.

Note: See TracTickets for help on using tickets.
Back to Top