Comment 2 for bug 352965

Revision history for this message
Stuart Bishop (stub) wrote :

Salgado and bac have also been looking into this:

https://pastebin.canonical.com/17460/

Newer Storm doesn't register Stores with the transaction manager until they are actually used, so transaction.abort() doesn't call Store.rollback() on these Stores. This is good, since it stops noop ROLLBACK statements being issued to the database. However, it breaks some tests apparently - probably sniffing to confirm that commits or rollbacks occur when they are expected to. We need to fix these tests, or apply the join transaction patch above to SQLBase._reset_store if we can confirm that method is only called by the test suite.