On Aug 24, 2009, at 6:08 PM, Steven Jenkins wrote: > > Luke Kanies wrote: > ... >> Is it possible to use the 'mail_patches' target to email all of this? >> It's much easier to comment on the code on the list. >> >> Unless you don't think it's ready for review yet... >> > > > I've pushed those to the list now.
My goal is to review this tomorrow. > > We definitely would like feedback before we go much further. The > Publisher and > Subscriber interfaces have jelled. The Event component still has > some work to > do (Ethan did some work on that which I have yet to pull in). > > My expectation (goal?) is to finish up the Event component and start > in on the > integration tomorrow, and now that we've got a lot of the initial > work done, I > expect things to move along at a more rapid pace. > > Of course, ripping out the Event pieces and replacing them in the > Puppet::Resource::Catalog, Puppet::Transaction*, Puppet::Type, etc > components, > is fairly hairy. What is your intuition as to which pieces would be > easiest to > tackle first? I clearly need to look at the code more closely, but I think everything except the Transaction bits should be pretty easy, and really, once you have a decent pub/sub system that supports our model (queues that collect events over time), even it should be relatively straightforward. And unfortunately, I'm not really sure you can do just one piece, because it all depends on the event queues, and once you change their usage, the whole thing breaks. -- The most incomprehensible thing about the world is that it is at all comprehensible. --Albert Einstein --------------------------------------------------------------------- Luke Kanies | http://reductivelabs.com | http://madstop.com --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en -~----------~----~----~----~------~----~------~--~---
