Hi Brett, Great work, I like it and I'm ok to merge it regularly to trunk.
I don't have idea about maps of beans. I think this branch is a good start to do more refactoring. 1- split DefaultBuildController to a controller + a builder The builder should be in it's own module and doesn't use/know the store, it checkout/update the working copy and build the project The controller get an event from the UI, the scheduler or an other way and send and event to one or more builder (local or remote), JMS is probably the solution 2- for each steps, the builder send the new project state notification, by JMS too and the Controller store it in memory, The project state must be transient instead of to store it in the db like today. When the build is done, the builder send a new event with the build result I have a point that I don't know for the moment how to resolve. How to access to the working copy when the builder is a remote builder and how to do when we use more than one remote builder? But we'll can find a solution later. Emmanuel On Tue, May 6, 2008 at 3:37 AM, Brett Porter <[EMAIL PROTECTED]> wrote: > ok, fixed and committed :) > > Anyone able to take a look through? > > Cheers, > Brett > > > On 02/05/2008, at 2:53 PM, Brett Porter wrote: > > me too after cleaning up. Sorry about that, I'll look into it. > > > > - Brett > > > > On 30/04/2008, at 5:52 AM, Olivier Lamy wrote: > > > > Hi, > > > I can't build the branch : > > > > > > /local/olamy/open-source/continuum-svn/builder-branch/continuum-core/src/main/java/org/apache/maven/continuum/core/action/AddProjectToCheckOutQueueAction.java:[60,16] > > > cannot find symbol > > > symbol : class CheckOutTask > > > location: class > > > org.apache.maven.continuum.core.action.AddProjectToCheckOutQueueAction > > > > > > > > > 2008/4/27 Brett Porter <[EMAIL PROTECTED]>: > > > > > > > Hi, > > > > > > > > I've started to do some refactoring - this is along the way to the > > > > builder > > > > separation I mentioned earlier. > > > > > > > > If you have a moment, please review r 651947. It's on a branch, but > > > > I'd > > > > like to regularly merge to trunk if there are no objects to avoid > > > > getting > > > > too distant. I have more tests to write for this first. > > > > > > > > All I've done is pulled the DefaultContinuumScm class out into a > > > > separate > > > > module, and decoupled it from the model. It already contained some > > > > logic > > > > related to the SCMs (which may actually need to go back into Maven > > > > SCM > > > > itself). I pulled the logging and database updates back into the > > > > actions in > > > > code (this did result in some duplication, but I can clean that up > > > > later). > > > > It also showed that some code was never having its results used, and > > > > also > > > > started to expose some exception handling bugs. I stopped wrapping > > > > exceptions and results, choosing to use the Maven SCM API natively. > > > > > > > > Thoughts? > > > > > > > > Anyone that is knowledgable in Spring, please check my work :) Is > > > > there a > > > > way to easily populate maps of beans, instead of hard coding the > > > > providers? > > > > > > > > > > > hehe it looks plexus have some nice features ;-) > > > > > > Cheers, > > > > Brett > > > > > > > > -- > > > > Brett Porter > > > > [EMAIL PROTECTED] > > > > http://blogs.exist.com/bporter/ > > > > > > > > > > > > > > -- > > Brett Porter > > [EMAIL PROTECTED] > > http://blogs.exist.com/bporter/ > > > > > -- > Brett Porter > [EMAIL PROTECTED] > http://blogs.exist.com/bporter/ > >
