I'd be happy with just adding prepare() as you describe and modifying commit() and rollback().
The default should certainly be off, as having it on by default is slow and dangerous if you don't have procedures in place to clean up prepared but uncommitted transactions. Extending this beyond that with begin() or explicit tpc_commit()/tpc_rollback() methods would warrant some discussion on the mailing list. In particular, you would also need to expose the connection.xid() method to allow people to generate the transaction ids . There is also the option of just adding the transaction id as an argument to prepare() if people need to set it explicitly. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/132485 Title: Storm should try to support two-phase commit where possible To manage notifications about this bug go to: https://bugs.launchpad.net/landscape/+bug/132485/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
