Hi, I've raised https://code.djangoproject.com/ticket/19918 and opened pull request: https://github.com/django/django/pull/852
And s/but/bug/ in my initial post :) Regards, Alex On Wednesday, February 20, 2013 9:58:36 AM UTC+2, Russell Keith-Magee wrote: > > Hi Ioan, > > I'd need to dig into the code to be 100% certain, but what you've > described seems plausible, and the fix looks like it's going in the right > direction. A ticket is certainly called for. > > As for the patch -- it needs tests :-) > > Django's test suite has support for testing multiple-database situations; > check the regressiontests/multiple_database test suite for examples of how > to use it. SELECT_FOR_UPDATE has it's own test suite - > modeltests/select_for_update - so the tests probably belong in there. > > Yours, > Russ Magee %-) > > -- 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 [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/django-developers?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
