[ mx4j-Bugs-1016117 ] Signature for javax.management package

Bugs item #1016117, was opened at 2004-08-25 18:26
Message generated for change (Comment added) made by biorn_steedom
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450647&aid=1016117&group_id=47745

Category: JMX implementation
Group: Release 2.0.1
Status: Open
Resolution: None
Priority: 5
Submitted By: Florent BENOIT (fbenoit)
Assigned to: Nobody/Anonymous (nobody)
Summary: Signature for javax.management package

Initial Comment:
	Hi,

Signature of JMX api (inside Mx4J) differs from the JSR
160 api
Mx4j is included in JOnAS (http://jonas.objectweb.org).

To be fully J2EE compliant, the signatures of
javax.management packages should be compliant too with
the J2EE specification.

For example :

For the class : javax.management.MBeanFeatureInfo

http://java.sun.com/j2ee/1.4/docs/api/javax/management/MBeanFeatureInfo.html
It should only implements Serializable but Mx4J class
implements both Serializable and Cloneable :
http://cvs.sourceforge.net/viewcvs.py/mx4j/mx4j/src/core/javax/management/MBeanFeatureInfo.java?rev=1.5&view=auto


javax.management.AttributeChangeNotificationFilter
should implements both NotificationFilter and
Serializable interfaces
http://java.sun.com/j2ee/1.4/docs/api/javax/management/AttributeChangeNotificationFilter.html
even if NotificationFilter class extends Serializable.


The classes
javax.management.modelmbean.ModelMBeanAttributeInfo,
javax.management.modelmbean.ModelMBeanConstructorInfo,
javax.management.modelmbean.ModelMBeanNotificationInfo
should implement javax.management.DescriptorAccess and
java.lang.Cloneable
http://java.sun.com/j2ee/1.4/docs/api/javax/management/modelmbean/ModelMBeanAttributeInfo.html
(even if superclass MBeanAttributeInfo already
implements Serializable).



javax.management.modelmbean.ModelMBeanInfo should not
implement Cloneable interface
http://cvs.sourceforge.net/viewcvs.py/mx4j/mx4j/src/core/javax/management/modelmbean/ModelMBeanInfo.java?rev=1.2&view=auto
http://java.sun.com/j2ee/1.4/docs/api/javax/management/modelmbean/ModelMBeanInfo.html
Also, it should not contain the method getConstructor()


javax.management.modelmbean.ModelMBeanInfoSupport
should implement both ModelMBeanInfo and Serializable
interface 
http://java.sun.com/j2ee/1.4/docs/api/javax/management/modelmbean/ModelMBeanInfoSupport.html

javax.management.modelmbean.RequiredModelMBean has a
wrong signature method for
removeAttributeChangeNotificationListener(javax.management.NotificationListener,java.lang.String)
public void
removeAttributeChangeNotificationListener(NotificationListener
listener, String attributeName) throws
RuntimeOperationsException, ListenerNotFoundException
should also throw MBeanException
http://java.sun.com/j2ee/1.4/docs/api/javax/management/modelmbean/RequiredModelMBean.html#removeAttributeChangeNotificationListener(javax.management.NotificationListener,%20java.lang.String)

I wonder if you could change these signatures, need a
patch or something else in order to be 100% compliant
with API.

Regards,

Florent.

JOnAS project.

----------------------------------------------------------------------

>Comment By: Simone Bordet (biorn_steedom)
Date: 2004-08-26 18:50

Message:
Logged In: YES 
user_id=128193

What do you mean with "fully J2EE compliant" ? Are you
running the JMX TCK against MX4J ?
If so, I hope Sun will fix the TCK bugs or at least ignore
the failures and grant compliance.
As for the MX4J bugs, I'm going to fix them, they're trivial
changes.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450647&aid=1016117&group_id=47745


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285