I have just committed a patch with a StateManageable interface and an AbstractStateManageable class.
Component now just has the methods to link it to a container (I wish the Container had methods to iterate over it's components!)
I have also taken some of Gianny's patch to put in some support for notification. I have not put the actual notification classes themselves in - as I'm not sure what we are doing in that space. So Gianny, you will have to find somebody who knows more about that bit to get the rest of your patch handled.
cheers
Sean Hamblett wrote:
Gianny,
The JSR77 Docs can be found here: http://jcp.org/aboutJava/communityprocess/final/jsr077/index.html. In my research of these docs, I understand the JSR is broken down into three main areas, State, Performance, and Events. At this time the Stat portion is somewhat implemented via the Component interface. I would like to see the StateManageable interface seperated out from the Component interface and have the State portion of JSR77 implemented in its own scope. That way the Container and/or the Component can implement this individually. If it is decided that the Container doesn't need to be a component, then the AbstractContainer interface can implement both Container, and StateManageable (via JSR77) and we can separate out the management duties from the container duties. Just my two bits.
Sean
On Wed, 13 Aug 2003 16:16:58 +0200 "gianny DAMOUR" <[EMAIL PROTECTED]> wrote:
Hi,
A small issue related to the org.apache.geronimo.common.State component is fixed by the enclosed patch (patch-core-java).
Moreover, the proposed patch adds the following components:
- org.apache.geronimo.common.EventMediator: Manage the event distribution among the management applications;
- org.apache.geronimo.common.EventProvider: Contained by an AbstractComponent. If the component is a "State Manageable Object", SMO, then it contains an EventProvider in order to broadcast some specific events;
- org.apache.geronimo.common.SMONotification: State event broadcasted by a SMO on entering of a specific event; and
- org.apache.geronimo.ejb.ContainerImplStateTest: Basic unit test registering a dummy management application tracking the state of a org.apache.geronimo.ejb.ContainerImpl component. As a matter of fact, ContainerImpl is not a component to be monitored according to the JSR77. However, it could be a good starting point to reflect on the proposed patch and decide how we want to provide the hooks required by JSR77.
org.apache.geronimo.ejb.ContainerImplStateTest (contained by patch-core-test) should be added to the geronimo\incubator\modules\core\src\test sub-project.
The other ones should be added to the geronimo\incubator\modules\core\src\java sub-project.
By the way, is it possible to add to the Wiki the exact procedure to follow to propose a patch?
A subsidiary question: where can I download the source of the javax.management.j2ee interfaces (JSR77)?
Cheers, Gianny
_________________________________________________________________ Trouvez l'�me soeur sur MSN Rencontres http://g.msn.fr/FR1000/9551
-- Greg Wilkins<[EMAIL PROTECTED]> Phone/fax: +44 7092063462 Mort Bay Consulting Australia and UK. http://www.mortbay.com
