I've left some comments on the PR - the overall approach is fine, and I'm in favour of things like removing quote_parameter and adding params to the deferred_sql entries - but there's a few stylistic questions and I'm worried why you've removed the maximum index name length stuff.
As for not being able to add autoincrement to an existing column - I'd say it's fine for you to state it's not supported and just raise a hard error when it happens. The situation is very rare and I doubt a lot of people will run into it. Andrew On Sat, Jan 4, 2014 at 4:29 PM, Michael Manfre <mman...@gmail.com> wrote: > I've been working on making django-mssql support the schema migration > changes that will be part of Django 1.7 and have a pull request [1] that is > mostly complete. Before I continue down my current path, I'd like to > solicit some feedback to help ensure my changes will get merged. > > I am currently able to pass all but 2 of the schema and migrations tests. > The most challenging failure is due to MSSQL's limitations with regards to > adding autoincrement to an existing column. The usual approach for doing > this sort of migration requires swapping in a new table with the existing > data, and fixing up all of the foreign keys. I may just document as "not > supported, use a commercial product, such as RedGate's SQLcompare, or make > the schema changes manually". > > For reference to see how I plan to use the API changes from my pull > request, see django-mssql's schema.py [2]. > > [1] https://github.com/django/django/pull/2143 > > [2] > https://bitbucket.org/Manfre/django-mssql/src/29c4d4ecc011cab0dcecadde6b7fe5f5e50bbf36/sqlserver_ado/schema.py?at=default > > Regards, > Michael Manfre > > -- > You received this message because you are subscribed to the Google Groups > "Django developers" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to django-developers+unsubscr...@googlegroups.com. > To post to this group, send email to django-developers@googlegroups.com. > Visit this group at http://groups.google.com/group/django-developers. > To view this discussion on the web visit > https://groups.google.com/d/msgid/django-developers/CAGdCwBu7dkBLk-kunGvuovGqTvwkOLfBzJ1B5AwpWqFyR-xLeg%40mail.gmail.com > . > For more options, visit https://groups.google.com/groups/opt_out. > -- You received this message because you are subscribed to the Google Groups "Django developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To post to this group, send email to django-developers@googlegroups.com. Visit this group at http://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/CAFwN1uprQZ%3DSxpocsgqVVPzS8r9LZDPQSQR7jZ9sgnCmNvqPNA%40mail.gmail.com. For more options, visit https://groups.google.com/groups/opt_out.
