#456 ManagementBean fixes

CVS HEAD
closed-rejected
JBossMX (22)
7
2004-01-30
2004-01-15
No

Here's a patch containing fixes to
org.jboss.management.mejb.ManagementBean to comply
with JMX 1.2.

ManagementBean.LocalConnector has been changed to
implement javax.management.MBeanServerConnection
instead of javax.management.MBeanServer.

The following classes have been updated to reflect this
change in implementation:
org.jboss.management.mejb.ClientNotificationListener
org.jboss.management.mejb.JMSClientNotificationListener
org.jboss.management.mejb.PollingClientNotificationListener
org.jboss.management.mejb.RMIClientNotificationListener
javax.management.j2ee.Management

Discussion

  • Charlton Barreto

    Logged In: YES
    user_id=456542

    To install, unzip or unjar this archive in ./jboss-head. It will
    overwrite the patched files in ./jboss-head/management.

     
  • Charlton Barreto

    • status: open --> open-fixed
     
  • Charlton Barreto

    • priority: 5 --> 7
     
  • Scott M Stark

    Scott M Stark - 2004-01-30
    • assigned_to: nobody --> starksm
    • status: open-fixed --> closed-rejected
     
  • Scott M Stark

    Scott M Stark - 2004-01-30

    Logged In: YES
    user_id=175228

    The J2EE 1.4 definition of the
    javax.management.j2ee.Management interface has not been
    updated to change the throws clause to use
    java.io.IOException instead of java.rmi.RemoteException:

    http://java.sun.com/j2ee/1.4/docs/api/index.html

    Thus the suggested change is not backward change as this
    code block will no longer compile:

    try
    {
    Management mejb = ...
    mejb.getAttribute(...);
    }
    catch(RemoteException e)
    {
    ...
    }

    Personally, I don't see the point of having an ejb delegate
    to another transport facade on top of the MBeanServer. The ejb
    already has the transport aspect incorporated into its
    service definition, so it should just be talking directly to the
    local MBeanServer. I recently removed the LocalConnector
    altogether.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks