geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jeremy Boynes" <jer...@coredevelopers.net>
Subject RE: [PATCH] JSR77 - 4 managed objects
Date Thu, 14 Aug 2003 22:56:47 GMT
Can you change a couple of things here please.

These should be interface definitions rather than concrete classes - there
could be a variety of classes implementing them.

Secondly, there should not need to be any references to javax.management in
them - the intention is that this model can be used by non-Java clients and
so all the attributes should be simple types; for example, the spec defines
the OBJECT_NAME type to be a string.

If you can fix these issues, I'll commit them.

Thanks
Jeremy

> -----Original Message-----
> From: gianny DAMOUR [mailto:gianny_damour@hotmail.com]
> Sent: Thursday, August 14, 2003 7:07 AM
> To: geronimo-dev@incubator.apache.org
> Subject: [PATCH] JSR77 - 4 managed objects
>
>
> Hi,
>
>
> The following patch implements - partially - the following model
> components
> of JSR77:
>
> - J2EEManagedObject;
> - J2EEDomain;
> - J2EEServer; and
> - JVM
>
> Two conveniance classes, namely J2EEModelElement and J2EEModelVisitor are
> defined to traverse the JSR77 model (Visitor pattern).
> J2EEModelVisitorTest
> is a unit test of the traversal mechanism.
>
> These components are defined in the package
> org.apache.geronimo.management.j2ee as suggested.
>
> It is definitively a small patch (4 components) compared to the number of
> components defined by JSR77 (32). However, it should be great to
> add it to
> CVS in order to keep others aware of the progress.
>
> Moreover, it should be great to create a new sub-project for JSR77, e.g.
> management, to keep the core sub-project browsable.
>
> Cheers,
> Gianny
>
> _________________________________________________________________
> The new MSN 8: advanced junk mail protection and 2 months FREE*
> http://join.msn.com/?page=features/junkmail
>


Mime
View raw message