cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Darren Shepherd <darren.s.sheph...@gmail.com>
Subject [MERGE] txn-refactor
Date Wed, 16 Oct 2013 16:59:00 GMT
I need as many people as possible to review this branch.  I'm still
testing it out, but I wanted to get as many eyes on it as possible.
This is a huge cross cutting change.  This branch is the changes to
use a new Transaction API that will be consistent Spring TX's style so
that we can eventually move to it.  You can get a bit more context
from https://cwiki.apache.org/confluence/display/CLOUDSTACK/Database+Transactions

Having spent so much time looking at the transaction management in
ACS, I'm well convinced we need to adopt Spring TX as soon as we can.
I've found just too many bugs.  It will be a painful transitiion, as
you can see from this branch.  It will also be very tricky, but I'll
figure it out.

If you are reviewing this branch, use a diff tool that ignores
whitespace.  Also if you don't know about "git difftool -d", you
should use that.  Just know, its was 10x more tedious and painful for
me to make this change then it is for you to review it :)

Darren

Mime
View raw message