On 14/06/11 12:02, Federico Di Gregorio wrote:
>> Thoughts?
> Yes, it is wrong. set_isolation_level() was wrong from the start but we
> had to keep it that way to avoid breaking compatibility. The discussion
> you linked still holds so I won't go back to sending a magic ROLLBACK.
>
> If we don't want to wait for the #@§%£$! person that wrote the Django
> code to fix it we can just make .autocommit a real attribute and set the
> session autocommit mode when it is assigned True/False.
Ok, I am completely #@§%£$! like a Django developer today (in fact I am
working on a project using it...).
Rewind.
Yes, it is wrong. [Rest of paragraph above...]
I think there is no way to fix this on our side. Even if we just ad an
attribute, like Karsten suggested Django code should still check it but
if they change the code they can just fix the problem without the need
to add an attribute.
federico
--
Federico Di Gregorio federico.digregorio@dndg.it
Studio Associato Di Nunzio e Di Gregorio http://dndg.it
The devil speaks truth much oftener than he's deemed.
He has an ignorant audience. -- Byron (suggested by Alice Fontana)