-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jürgen kartnaller wrote: > I just run into a with transaction manager and synchronizers. > > If transaction.get implicitily creates a new transaction the synchronizers > are not called. This is not problem as long as no savepoints are used. If > savepoints are used the DataManagers created by synchronizers are not > present. Because savepoints are requested from the DataManagers it doesn't > work.
Can you elaborate on how your patch interacts with this comment (in transaction._manager)? # Call the ISynchronizer newTransaction() method on every element of # WeakSet synchs. # A transaction manager needs to do this whenever begin() is called. # Since it would be good if tm.get() returned the new transaction while # newTransaction() is running, calling this has to be delayed until # after # the transaction manager has done whatever it needs to do to make its # get() return the new txn. In other words, is there any cases there TM.get() will be called when it is "too early" to call '_new_transaction'? > I already created a branch with a fix (transaction/branches/synchfix). > > If one with more knowledge about the transaction manager can approve my > change I will add tests and merge the change into trunk. It would help to see the tests with the rest of the patch: they often help illustrate the motivation. Tres. - -- =================================================================== Tres Seaver +1 540-429-0999 [EMAIL PROTECTED] Palladion Software "Excellence by Design" http://palladion.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFI15wu+gerLs4ltQ4RAiA8AJ9hLInfO/5c5633rgqY784YdBjsTgCgoMjM gAA7c5VH4jhy+wTVt+uXztk= =rMB8 -----END PGP SIGNATURE----- _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
