Nico Klasens wrote:
[...]
I made the changes on the 1.7 branch, because that is now running here in production. The changes can easily be applied to the HEAD.
I would like to add the changes to 1.7 and head. The code of the multicast is for 95% the same (code only moved around for readability and easy extension).
Committors: please choose one of the options below.
START OF CALL: 2004-09-22 END OF CALL: 2004-09-27
[_] +1 (YES) [_] +0 (ABSTAIN ) [_] -1 (NO), because : [_] VETO, because:
Despite the fact that most of the code is the same, I think this change is too big for 1.7.x. Maybe it can be added to cvs head and after a while (if people are willing to test it in cvs-head) it can be backported.
I'm also not sure whether it's a good idea for cvs-head, because a lot of discussion lately is about multicast/notification and whether a complete new pluggable solution is needed.
Maybe we can add this to cvs-head for the time being and in the near future start a project for making it more pluggable and look into the whole event/notification structure in MMBase.
I'm not sure whether the packagename must be something under storage. I like org.mmbase.notification (without the storage).
Gerard
