Nabble - OpenJDK Java Management Extensions Development

Re: jmx-dev Need reviewer for 6888171: JMX Monitor API should not require JavaBeans to be present

2009-11-23T05:30:30Z

Looks OK to me. There's a typo "upperacse" in Introspector but that's all I noticed.

Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus
<http://weblogs.java.net/blog/emcmanus/>

Alan Bateman wrote:

>
> The JMX Monitor API is specified to use the Java Beans introspector for
> complex types others than CompositeData or arrays. This dependency is
> undesirable because JavaBeans is very tied to AWT. I'd like to address
> this dependency. The main observation is that if explicit information is
> available (ie: XXXBeanInfo exists) then the application's dependency
> will require Java Beans to be present. In that case, the Java Beans
> introspector should be used and everywhere works as it does now. On the
> other hand, if explicit information is not available, then the
> introspector uses reflection to identity the property accessors. That
> case should not require the module containing Java Beans to be installed
> and if not installed, we can use a fallback mechanism to identity the
> property methods. I had a short call with Éamonn about this and he
> agreed that this approach is okay (and made useful suggestions to avoid
> duplicating some of the existing JMX introspection code). I've put the
> webrev the proposed changes here:
> http://cr.openjdk.java.net/~alanb/6888171/webrev.00
>
> Thanks,
> Alan.

jmx-dev Need reviewer for 6888171: JMX Monitor API should not require JavaBeans to be present

2009-11-23T02:04:26Z

The JMX Monitor API is specified to use the Java Beans introspector for
complex types others than CompositeData or arrays. This dependency is
undesirable because JavaBeans is very tied to AWT. I'd like to address
this dependency. The main observation is that if explicit information is
available (ie: XXXBeanInfo exists) then the application's dependency
will require Java Beans to be present. In that case, the Java Beans
introspector should be used and everywhere works as it does now. On the
other hand, if explicit information is not available, then the
introspector uses reflection to identity the property accessors. That
case should not require the module containing Java Beans to be installed
and if not installed, we can use a fallback mechanism to identity the
property methods. I had a short call with Éamonn about this and he
agreed that this approach is okay (and made useful suggestions to avoid
duplicating some of the existing JMX introspection code). I've put the
webrev the proposed changes here:
http://cr.openjdk.java.net/~alanb/6888171/webrev.00

Thanks,
Alan.

Re: jmx-dev Need reviewer for 6888179: Separate out dependency on CORBA

2009-10-26T04:38:26Z

Hi Alan,

This looks OK to me.

Cheers,

-- daniel
JMX, SNMP, Java, etc... http://blogs.sun.com/jmxetc

Alan Bateman wrote:

>
> As you know, we need to do some re-organization of code in preparation
> for the jdk build generating modules. One dependency that I would like
> to separate out is the dependency on RMI-IIOP and CORBA in the JSR-160
> implementation. The motivation is a possible "management" module that
> wouldn't require "corba" to be installed. The webrev with the proposed
> changes is here:
> http://cr.openjdk.java.net/~alanb/6888179/webrev.00/
>
> In summary, the RMI connectors are changed to use a helper class for
> RMI-IIOP and CORBA. The methods defined by the helper class have all
> CORBA types erased and they simply delegate to a proxy which has an
> implementation in the com.sun.jmx.remote.protocol.iiop package that is
> either present or not present. When not present then methods requiring
> the transport fail gracefully indicating that the iiop protocol is not
> supported.
>
> I ran into two issues that I should point:
>
> 1. The bytecode for ProxyStub._releaseReply in RMIConnector didn't match
> the source in the comments. The actual code has a null check that was
> missing from the source. I've updated this for any future maintainers.
>
> 2. RMIConnector.findRMIServerJRMP used PortableRemoteObject.narrow to
> check the stub type, which I'm sure was an oversight.
>
> I've run all the tests in test/javax/management/remote (good test
> coverage btw) and don't see any regressions.
>
> Éamonn, Daniel - would you have cycles to review?
>
> -Alan.

jmx-dev [Fwd: hg: jdk7/tl/jdk: 6851617: Remove JSR 255 (JMX API 2.0) from JDK 7]

2009-10-21T08:50:21Z

This took longer than I expected, but as I explained on my blog, JSR 255 is being decommitted from JDK 7 but will live on in a new repository until it can be integrated into a later JDK release.

--
Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus

Changeset: f23a3ae59169
Author: emcmanus
Date: 2009-10-21 17:33 +0200
URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/f23a3ae59169

6851617: Remove JSR 255 (JMX API 2.0) from JDK 7
Summary: See http://weblogs.java.net/blog/2009/06/16/jsr-255-jmx-api-20-postponed
Reviewed-by: dfuchs

! make/docs/CORE_PKGS.gmk
! src/share/classes/com/sun/jmx/defaults/JmxProperties.java
! src/share/classes/com/sun/jmx/defaults/ServiceName.java
- src/share/classes/com/sun/jmx/event/DaemonThreadFactory.java
- src/share/classes/com/sun/jmx/event/EventBuffer.java
- src/share/classes/com/sun/jmx/event/EventClientFactory.java
- src/share/classes/com/sun/jmx/event/EventConnection.java
- src/share/classes/com/sun/jmx/event/EventParams.java
- src/share/classes/com/sun/jmx/event/LeaseManager.java
- src/share/classes/com/sun/jmx/event/LeaseRenewer.java
- src/share/classes/com/sun/jmx/event/ReceiverBuffer.java
- src/share/classes/com/sun/jmx/event/RepeatedSingletonJob.java
! src/share/classes/com/sun/jmx/interceptor/DefaultMBeanServerInterceptor.java
- src/share/classes/com/sun/jmx/interceptor/DispatchInterceptor.java
- src/share/classes/com/sun/jmx/interceptor/DomainDispatchInterceptor.java
- src/share/classes/com/sun/jmx/interceptor/MBeanServerInterceptorSupport.java
- src/share/classes/com/sun/jmx/interceptor/NamespaceDispatchInterceptor.java
- src/share/classes/com/sun/jmx/interceptor/SingleMBeanForwarder.java
! src/share/classes/com/sun/jmx/mbeanserver/ConvertingMethod.java
! src/share/classes/com/sun/jmx/mbeanserver/DefaultMXBeanMappingFactory.java
! src/share/classes/com/sun/jmx/mbeanserver/DynamicMBean2.java
! src/share/classes/com/sun/jmx/mbeanserver/Introspector.java
! src/share/classes/com/sun/jmx/mbeanserver/JmxMBeanServer.java
! src/share/classes/com/sun/jmx/mbeanserver/MBeanAnalyzer.java
- src/share/classes/com/sun/jmx/mbeanserver/MBeanInjector.java
! src/share/classes/com/sun/jmx/mbeanserver/MBeanInstantiator.java
! src/share/classes/com/sun/jmx/mbeanserver/MBeanIntrospector.java
! src/share/classes/com/sun/jmx/mbeanserver/MBeanSupport.java
! src/share/classes/com/sun/jmx/mbeanserver/MXBeanIntrospector.java
! src/share/classes/com/sun/jmx/mbeanserver/MXBeanLookup.java
+ src/share/classes/com/sun/jmx/mbeanserver/MXBeanMapping.java
+ src/share/classes/com/sun/jmx/mbeanserver/MXBeanMappingFactory.java
! src/share/classes/com/sun/jmx/mbeanserver/MXBeanProxy.java
! src/share/classes/com/sun/jmx/mbeanserver/MXBeanSupport.java
- src/share/classes/com/sun/jmx/mbeanserver/NotificationMBeanSupport.java
- src/share/classes/com/sun/jmx/mbeanserver/NotifySupport.java
! src/share/classes/com/sun/jmx/mbeanserver/PerInterface.java
- src/share/classes/com/sun/jmx/mbeanserver/PerThreadGroupPool.java
! src/share/classes/com/sun/jmx/mbeanserver/Repository.java
! src/share/classes/com/sun/jmx/mbeanserver/StandardMBeanIntrospector.java
! src/share/classes/com/sun/jmx/mbeanserver/StandardMBeanSupport.java
! src/share/classes/com/sun/jmx/mbeanserver/Util.java
- src/share/classes/com/sun/jmx/namespace/DomainInterceptor.java
- src/share/classes/com/sun/jmx/namespace/HandlerInterceptor.java
- src/share/classes/com/sun/jmx/namespace/NamespaceInterceptor.java
- src/share/classes/com/sun/jmx/namespace/ObjectNameRouter.java
- src/share/classes/com/sun/jmx/namespace/RoutingConnectionProxy.java
- src/share/classes/com/sun/jmx/namespace/RoutingMBeanServerConnection.java
- src/share/classes/com/sun/jmx/namespace/RoutingProxy.java
- src/share/classes/com/sun/jmx/namespace/RoutingServerProxy.java
- src/share/classes/com/sun/jmx/namespace/package.html
- src/share/classes/com/sun/jmx/namespace/serial/DefaultRewritingProcessor.java
- src/share/classes/com/sun/jmx/namespace/serial/IdentityProcessor.java
- src/share/classes/com/sun/jmx/namespace/serial/JMXNamespaceContext.java
- src/share/classes/com/sun/jmx/namespace/serial/RewritingProcessor.java
- src/share/classes/com/sun/jmx/namespace/serial/RoutingOnlyProcessor.java
- src/share/classes/com/sun/jmx/namespace/serial/SerialRewritingProcessor.java
- src/share/classes/com/sun/jmx/namespace/serial/package.html
! src/share/classes/com/sun/jmx/remote/internal/ServerNotifForwarder.java
! src/share/classes/com/sun/jmx/remote/util/EnvHelp.java
- src/share/classes/com/sun/jmx/remote/util/EventClientConnection.java
! src/share/classes/java/lang/management/PlatformComponent.java
! src/share/classes/java/util/logging/Logging.java
! src/share/classes/javax/management/AndQueryExp.java
! src/share/classes/javax/management/AttributeList.java
! src/share/classes/javax/management/AttributeValueExp.java
! src/share/classes/javax/management/BetweenQueryExp.java
! src/share/classes/javax/management/BinaryOpValueExp.java
! src/share/classes/javax/management/BinaryRelQueryExp.java
- src/share/classes/javax/management/ClientContext.java
- src/share/classes/javax/management/Description.java
! src/share/classes/javax/management/Descriptor.java
- src/share/classes/javax/management/DescriptorFields.java
! src/share/classes/javax/management/DescriptorKey.java
- src/share/classes/javax/management/DynamicWrapperMBean.java
- src/share/classes/javax/management/GenericMBeanException.java
- src/share/classes/javax/management/Impact.java
! src/share/classes/javax/management/InstanceNotFoundException.java
! src/share/classes/javax/management/JMRuntimeException.java
! src/share/classes/javax/management/JMX.java
- src/share/classes/javax/management/MBean.java
! src/share/classes/javax/management/MBeanAttributeInfo.java
! src/share/classes/javax/management/MBeanConstructorInfo.java
! src/share/classes/javax/management/MBeanException.java
! src/share/classes/javax/management/MBeanInfo.java
! src/share/classes/javax/management/MBeanOperationInfo.java
! src/share/classes/javax/management/MBeanPermission.java
! src/share/classes/javax/management/MBeanRegistration.java
! src/share/classes/javax/management/MBeanServer.java
! src/share/classes/javax/management/MBeanServerConnection.java
! src/share/classes/javax/management/MBeanServerDelegate.java
! src/share/classes/javax/management/MBeanServerFactory.java
! src/share/classes/javax/management/MBeanServerInvocationHandler.java
! src/share/classes/javax/management/MBeanServerNotification.java
! src/share/classes/javax/management/MXBean.java
- src/share/classes/javax/management/ManagedAttribute.java
- src/share/classes/javax/management/ManagedOperation.java
! src/share/classes/javax/management/NotQueryExp.java
! src/share/classes/javax/management/Notification.java
! src/share/classes/javax/management/NotificationBroadcasterSupport.java
- src/share/classes/javax/management/NotificationInfo.java
- src/share/classes/javax/management/NotificationInfos.java
! src/share/classes/javax/management/ObjectName.java
- src/share/classes/javax/management/ObjectNameTemplate.java
! src/share/classes/javax/management/OrQueryExp.java
! src/share/classes/javax/management/QualifiedAttributeValueExp.java
! src/share/classes/javax/management/Query.java
! src/share/classes/javax/management/QueryEval.java
- src/share/classes/javax/management/QueryNotificationFilter.java
- src/share/classes/javax/management/QueryParser.java
- src/share/classes/javax/management/SendNotification.java
! src/share/classes/javax/management/StandardEmitterMBean.java
! src/share/classes/javax/management/StandardMBean.java
- src/share/classes/javax/management/event/EventClient.java
- src/share/classes/javax/management/event/EventClientDelegate.java
- src/share/classes/javax/management/event/EventClientDelegateMBean.java
- src/share/classes/javax/management/event/EventClientNotFoundException.java
- src/share/classes/javax/management/event/EventConsumer.java
- src/share/classes/javax/management/event/EventForwarder.java
- src/share/classes/javax/management/event/EventReceiver.java
- src/share/classes/javax/management/event/EventRelay.java
- src/share/classes/javax/management/event/EventSubscriber.java
- src/share/classes/javax/management/event/FetchingEventForwarder.java
- src/share/classes/javax/management/event/FetchingEventRelay.java
- src/share/classes/javax/management/event/ListenerInfo.java
- src/share/classes/javax/management/event/NotificationManager.java
- src/share/classes/javax/management/event/RMIPushEventForwarder.java
- src/share/classes/javax/management/event/RMIPushEventRelay.java
- src/share/classes/javax/management/event/RMIPushServer.java
- src/share/classes/javax/management/event/package-info.java
! src/share/classes/javax/management/modelmbean/DescriptorSupport.java
! src/share/classes/javax/management/modelmbean/ModelMBeanAttributeInfo.java
! src/share/classes/javax/management/modelmbean/ModelMBeanConstructorInfo.java
! src/share/classes/javax/management/modelmbean/ModelMBeanInfoSupport.java
! src/share/classes/javax/management/modelmbean/ModelMBeanNotificationInfo.java
! src/share/classes/javax/management/modelmbean/ModelMBeanOperationInfo.java
! src/share/classes/javax/management/modelmbean/RequiredModelMBean.java
! src/share/classes/javax/management/monitor/MonitorNotification.java
- src/share/classes/javax/management/namespace/JMXDomain.java
- src/share/classes/javax/management/namespace/JMXNamespace.java
- src/share/classes/javax/management/namespace/JMXNamespaceMBean.java
- src/share/classes/javax/management/namespace/JMXNamespacePermission.java
- src/share/classes/javax/management/namespace/JMXNamespaceView.java
- src/share/classes/javax/management/namespace/JMXNamespaces.java
- src/share/classes/javax/management/namespace/JMXRemoteNamespace.java
- src/share/classes/javax/management/namespace/JMXRemoteNamespaceMBean.java
- src/share/classes/javax/management/namespace/MBeanServerConnectionWrapper.java
- src/share/classes/javax/management/namespace/MBeanServerSupport.java
- src/share/classes/javax/management/namespace/VirtualEventManager.java
- src/share/classes/javax/management/namespace/package-info.java
! src/share/classes/javax/management/openmbean/CompositeDataInvocationHandler.java
! src/share/classes/javax/management/openmbean/CompositeDataSupport.java
! src/share/classes/javax/management/openmbean/CompositeType.java
- src/share/classes/javax/management/openmbean/MXBeanMapping.java
- src/share/classes/javax/management/openmbean/MXBeanMappingClass.java
- src/share/classes/javax/management/openmbean/MXBeanMappingFactory.java
- src/share/classes/javax/management/openmbean/MXBeanMappingFactoryClass.java
! src/share/classes/javax/management/openmbean/OpenType.java
! src/share/classes/javax/management/package.html
- src/share/classes/javax/management/remote/IdentityMBeanServerForwarder.java
! src/share/classes/javax/management/remote/JMXConnector.java
! src/share/classes/javax/management/remote/JMXConnectorServer.java
! src/share/classes/javax/management/remote/JMXConnectorServerMBean.java
! src/share/classes/javax/management/remote/JMXServiceURL.java
! src/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
! src/share/classes/javax/management/remote/rmi/RMIConnector.java
! src/share/classes/javax/management/remote/rmi/RMIConnectorServer.java
! src/share/classes/javax/management/remote/rmi/RMIServerImpl.java
! src/share/classes/sun/management/ClassLoadingImpl.java
! src/share/classes/sun/management/CompilationImpl.java
! src/share/classes/sun/management/HotSpotDiagnostic.java
! src/share/classes/sun/management/HotspotInternal.java
! src/share/classes/sun/management/ManagementFactoryHelper.java
! src/share/classes/sun/management/MemoryImpl.java
! src/share/classes/sun/management/OperatingSystemImpl.java
! src/share/classes/sun/management/RuntimeImpl.java
! src/share/classes/sun/management/ThreadImpl.java
! src/share/classes/sun/management/Util.java
- test/javax/management/Introspector/AnnotatedMBeanTest.java
- test/javax/management/Introspector/AnnotatedNotificationInfoTest.java
! test/javax/management/Introspector/AnnotationTest.java
- test/javax/management/Introspector/ExceptionsDescriptorTest.java
- test/javax/management/Introspector/MBeanDescriptionTest.java
- test/javax/management/Introspector/ObjectNameTemplateTest.java
- test/javax/management/Introspector/ParameterNameTest.java
- test/javax/management/Introspector/ResourceInjectionTest.java
- test/javax/management/MBeanServer/AttributeListMapTest.java
- test/javax/management/MBeanServer/DynamicWrapperMBeanTest.java
- test/javax/management/MBeanServer/InstanceNotFoundExceptionTest.java
! test/javax/management/MBeanServer/MBeanExceptionTest.java
- test/javax/management/MBeanServerFactory/NamedMBeanServerTest.java
! test/javax/management/ObjectName/ApplyWildcardTest.java
- test/javax/management/ObjectName/ValueOfTest.java
- test/javax/management/context/ContextForwarderTest.java
- test/javax/management/context/ContextTest.java
- test/javax/management/context/LocaleAwareBroadcasterTest.java
- test/javax/management/context/LocaleTest.java
- test/javax/management/context/LocalizableTest.java
- test/javax/management/context/RemoteContextTest.java
- test/javax/management/context/localizable/MBeanDescriptions.properties
- test/javax/management/context/localizable/MBeanDescriptions_fr.java
- test/javax/management/context/localizable/Whatsit.java
- test/javax/management/context/localizable/WhatsitMBean.java
- test/javax/management/descriptor/DescriptorConstructorTest.java
- test/javax/management/eventService/AddRemoveListenerTest.java
- test/javax/management/eventService/CustomForwarderTest.java
- test/javax/management/eventService/EventClientExecutorTest.java
- test/javax/management/eventService/EventClientThreadTest.java
- test/javax/management/eventService/EventDelegateSecurityTest.java
- test/javax/management/eventService/EventManagerTest.java
- test/javax/management/eventService/FetchingTest.java
- test/javax/management/eventService/LeaseManagerDeadlockTest.java
- test/javax/management/eventService/LeaseTest.java
- test/javax/management/eventService/ListenerTest.java
- test/javax/management/eventService/MyFetchingEventForwarder.java
- test/javax/management/eventService/NotSerializableNotifTest.java
- test/javax/management/eventService/PublishTest.java
- test/javax/management/eventService/ReconnectableConnectorTest.java
- test/javax/management/eventService/SharingThreadTest.java
- test/javax/management/eventService/SubUnsubTest.java
- test/javax/management/eventService/SubscribeTest.java
- test/javax/management/eventService/UsingEventService.java
- test/javax/management/interop/MBeanExceptionInteropTest.java
- test/javax/management/modelmbean/DefaultDescriptorFieldTest.java
! test/javax/management/monitor/AttributeArbitraryDataTypeTest.java
- test/javax/management/monitor/InstantiateMonitorNotificationTest.java
- test/javax/management/mxbean/CustomTypeTest.java
! test/javax/management/mxbean/ExceptionDiagnosisTest.java
- test/javax/management/mxbean/JMXServiceURLTest.java
- test/javax/management/mxbean/customtypes/CustomLongMXBean.java
- test/javax/management/mxbean/customtypes/CustomMXBean.java
- test/javax/management/mxbean/customtypes/IntegerIsLongFactory.java
- test/javax/management/mxbean/customtypes/IntegerIsStringFactory.java
- test/javax/management/mxbean/customtypes/package-info.java
- test/javax/management/namespace/DomainCreationTest.java
- test/javax/management/namespace/EventWithNamespaceControlTest.java
- test/javax/management/namespace/EventWithNamespaceTest.java
- test/javax/management/namespace/ExportNamespaceTest.java
- test/javax/management/namespace/JMXDomainTest.java
- test/javax/management/namespace/JMXNamespaceSecurityTest.java
- test/javax/management/namespace/JMXNamespaceTest.java
- test/javax/management/namespace/JMXNamespaceViewTest.java
- test/javax/management/namespace/JMXNamespacesTest.java
- test/javax/management/namespace/JMXRemoteNamespaceTest.java
- test/javax/management/namespace/JMXRemoteTargetNamespace.java
- test/javax/management/namespace/LazyDomainTest.java
- test/javax/management/namespace/LeadingSeparatorsTest.java
- test/javax/management/namespace/MXBeanRefTest.java
- test/javax/management/namespace/NamespaceController.java
- test/javax/management/namespace/NamespaceControllerMBean.java
- test/javax/management/namespace/NamespaceCreationTest.java
- test/javax/management/namespace/NamespaceNotificationsTest.java
- test/javax/management/namespace/NullDomainObjectNameTest.java
- test/javax/management/namespace/NullObjectNameTest.java
- test/javax/management/namespace/QueryNamesTest.java
- test/javax/management/namespace/RemoveNotificationListenerTest.java
- test/javax/management/namespace/RoutingServerProxyTest.java
- test/javax/management/namespace/SerialParamProcessorTest.java
- test/javax/management/namespace/SourceNamespaceTest.java
- test/javax/management/namespace/VirtualMBeanNotifTest.java
- test/javax/management/namespace/VirtualMBeanTest.java
- test/javax/management/namespace/VirtualNamespaceQueryTest.java
- test/javax/management/namespace/VirtualPropsTest.java
- test/javax/management/namespace/Wombat.java
- test/javax/management/namespace/WombatMBean.java
- test/javax/management/namespace/namespace.policy
- test/javax/management/notification/SupportClearTest.java
- test/javax/management/openmbean/CompositeDataToMapTest.java
- test/javax/management/openmbean/GenericMBeanExceptionTest.java
- test/javax/management/query/QueryDottedAttrTest.java
! test/javax/management/query/QueryExpStringTest.java
- test/javax/management/query/QueryNotifFilterTest.java
- test/javax/management/query/QueryParseTest.java
! test/javax/management/remote/mandatory/connection/CloseServerTest.java
! test/javax/management/remote/mandatory/connection/DeadLockTest.java
! test/javax/management/remote/mandatory/connection/IdleTimeoutTest.java
! test/javax/management/remote/mandatory/connection/RMIExitTest.java
! test/javax/management/remote/mandatory/connection/ReconnectTest.java
- test/javax/management/remote/mandatory/connectorServer/CloseConnectionTest.java
- test/javax/management/remote/mandatory/connectorServer/ForwarderChainTest.java
- test/javax/management/remote/mandatory/connectorServer/StandardForwardersTest.java
! test/javax/management/remote/mandatory/loading/MissingClassTest.java
! test/javax/management/remote/mandatory/notif/AddRemoveTest.java
! test/javax/management/remote/mandatory/notif/DiffHBTest.java
! test/javax/management/remote/mandatory/notif/EmptyDomainNotificationTest.java
! test/javax/management/remote/mandatory/notif/ListenerScaleTest.java
! test/javax/management/remote/mandatory/notif/NotifBufferSizePropertyNameTest.java
! test/javax/management/remote/mandatory/notif/NotifReconnectDeadlockTest.java
! test/javax/management/remote/mandatory/notif/NotificationBufferCreationTest.java
! test/javax/management/remote/mandatory/notif/NotificationBufferDeadlockTest.java
! test/javax/management/remote/mandatory/notif/NotificationEmissionTest.java
! test/javax/management/remote/mandatory/notif/RMINotifTest.java
! test/javax/management/remote/mandatory/notif/UnexpectedNotifTest.java
- test/javax/management/remote/mandatory/version/JMXSpecVersionTest.java
- test/javax/management/standardmbean/FindMethodTest.java
- test/javax/management/standardmbean/RegistrationTest.java

jmx-dev Need reviewer for 6888179: Separate out dependency on CORBA

2009-10-21T03:23:44Z

As you know, we need to do some re-organization of code in preparation
for the jdk build generating modules. One dependency that I would like
to separate out is the dependency on RMI-IIOP and CORBA in the JSR-160
implementation. The motivation is a possible "management" module that
wouldn't require "corba" to be installed. The webrev with the proposed
changes is here:
http://cr.openjdk.java.net/~alanb/6888179/webrev.00/

In summary, the RMI connectors are changed to use a helper class for
RMI-IIOP and CORBA. The methods defined by the helper class have all
CORBA types erased and they simply delegate to a proxy which has an
implementation in the com.sun.jmx.remote.protocol.iiop package that is
either present or not present. When not present then methods requiring
the transport fail gracefully indicating that the iiop protocol is not
supported.

I ran into two issues that I should point:

1. The bytecode for ProxyStub._releaseReply in RMIConnector didn't match
the source in the comments. The actual code has a null check that was
missing from the source. I've updated this for any future maintainers.

2. RMIConnector.findRMIServerJRMP used PortableRemoteObject.narrow to
check the stub type, which I'm sure was an oversight.

I've run all the tests in test/javax/management/remote (good test
coverage btw) and don't see any regressions.

Éamonn, Daniel - would you have cycles to review?

-Alan.

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T06:27:40Z

In message http://mail.openjdk.java.net/pipermail/jmx-dev/2009-July/000050.html I wrote:
> Should the JMX Group sponsor the proposed JMX2 Project?

5 of the 8 Members eligible to vote have now voted Yes:
Jean-François Denise, Daniel Fuchs, Yves Joan, Sandra Lions, Éamonn McManus.
Since that constitutes an absolute majority, I have the pleasure as the Group's Moderator
of announcing that the JMX Group has decided to sponsor the JMX2 Project.

Regards,
Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus/

Eamonn McManus wrote:

> This is a Call For Votes (CFV) on the following question:
>
> Should the JMX Group sponsor the proposed JMX2 Project [1]?
>
> Please cast your vote by replying, publicly, to this message with either
>
> Vote: yes
>
> or
>
> Vote: no
>
> as the first line of the message body.
>
> You may, at your option, indicate the reason for your decision on
> subsequent lines.
>
> Votes must be cast in the open; votes sent as private replies will not
> be counted.
>
> The sponsorship decision will be made by a simple majority vote of the
> Group's Members.
> Votes are due by midnight UTC next Wednesday, July 22nd.
> As an optimization, if an absolute majority of the Group's Members votes
> one way or the other prior to that time then the decision may be
> rendered earlier.
>
> Only Members of the JMX Group are eligible to vote on this decision.
> The current Members are:
>
> Luis-Miguel Alventosa
> Jean-François Denise
> Joël Féraud
> Daniel Fuchs
> Shanliang Jiang
> Yves Joan
> Sandra Lions
> Éamonn McManus
>
> Once a decision has been made, the votes will be summarized and report
> to this list
> and also to discuss at openjdk.java.net.
>
> [1] http://mail.openjdk.java.net/pipermail/announce/2009-July/000080.html

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T04:32:11Z

Vote: yes

-- daniel

Eamonn McManus wrote:

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T02:46:45Z

Vote: yes
Yves

Eamonn McManus wrote:

> This is a Call For Votes (CFV) on the following question:
>
> Should the JMX Group sponsor the proposed JMX2 Project [1]?
>
> Please cast your vote by replying, publicly, to this message with either
>
> Vote: yes
>
> or
>
> Vote: no
>
> as the first line of the message body.
>
> You may, at your option, indicate the reason for your decision on
> subsequent lines.
>
> Votes must be cast in the open; votes sent as private replies will not
> be counted.
>
> The sponsorship decision will be made by a simple majority vote of the
> Group's Members.
> Votes are due by midnight UTC next Wednesday, July 22nd.
> As an optimization, if an absolute majority of the Group's Members
> votes one way or the other prior to that time then the decision may be
> rendered earlier.
>
> Only Members of the JMX Group are eligible to vote on this decision.
> The current Members are:
>
> Luis-Miguel Alventosa
> Jean-François Denise
> Joël Féraud
> Daniel Fuchs
> Shanliang Jiang
> Yves Joan
> Sandra Lions
> Éamonn McManus
>
> Once a decision has been made, the votes will be summarized and report
> to this list
> and also to discuss at openjdk.java.net.
>
> [1] http://mail.openjdk.java.net/pipermail/announce/2009-July/000080.html

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T02:17:23Z

Vote: yes

Eamonn McManus wrote:

> This is a Call For Votes (CFV) on the following question:
>
> Should the JMX Group sponsor the proposed JMX2 Project [1]?
>
> Please cast your vote by replying, publicly, to this message with either
>
> Vote: yes
>
> or
>
> Vote: no
>
> as the first line of the message body.
>
> You may, at your option, indicate the reason for your decision on
> subsequent lines.
>
> Votes must be cast in the open; votes sent as private replies will not
> be counted.
>
> The sponsorship decision will be made by a simple majority vote of the
> Group's Members.
> Votes are due by midnight UTC next Wednesday, July 22nd.
> As an optimization, if an absolute majority of the Group's Members
> votes one way or the other prior to that time then the decision may be
> rendered earlier.
>
> Only Members of the JMX Group are eligible to vote on this decision.
> The current Members are:
>
> Luis-Miguel Alventosa
> Jean-François Denise
> Joël Féraud
> Daniel Fuchs
> Shanliang Jiang
> Yves Joan
> Sandra Lions
> Éamonn McManus
>
> Once a decision has been made, the votes will be summarized and report
> to this list
> and also to discuss at openjdk.java.net.
>
> [1] http://mail.openjdk.java.net/pipermail/announce/2009-July/000080.html

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T02:01:21Z

Vote: yes

Sandra

Eamonn McManus wrote:

> This is a Call For Votes (CFV) on the following question:
>
> Should the JMX Group sponsor the proposed JMX2 Project [1]?
>
> Please cast your vote by replying, publicly, to this message with either
>
> Vote: yes
>
> or
>
> Vote: no
>
> as the first line of the message body.
>
> You may, at your option, indicate the reason for your decision on
> subsequent lines.
>
> Votes must be cast in the open; votes sent as private replies will not
> be counted.
>
> The sponsorship decision will be made by a simple majority vote of the
> Group's Members.
> Votes are due by midnight UTC next Wednesday, July 22nd.
> As an optimization, if an absolute majority of the Group's Members
> votes one way or the other prior to that time then the decision may be
> rendered earlier.
>
> Only Members of the JMX Group are eligible to vote on this decision.
> The current Members are:
>
> Luis-Miguel Alventosa
> Jean-François Denise
> Joël Féraud
> Daniel Fuchs
> Shanliang Jiang
> Yves Joan
> Sandra Lions
> Éamonn McManus
>
> Once a decision has been made, the votes will be summarized and report
> to this list
> and also to discuss at openjdk.java.net.
>
> [1] http://mail.openjdk.java.net/pipermail/announce/2009-July/000080.html

--
Sandra

Re: jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T01:56:31Z

Vote: yes

Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus/

Eamonn McManus wrote:

jmx-dev CFV: Project sponsorship: JMX2

2009-07-15T01:55:26Z

This is a Call For Votes (CFV) on the following question:

Should the JMX Group sponsor the proposed JMX2 Project [1]?

Please cast your vote by replying, publicly, to this message with either

Vote: yes

or

Vote: no

as the first line of the message body.

You may, at your option, indicate the reason for your decision on subsequent lines.

Votes must be cast in the open; votes sent as private replies will not be counted.

The sponsorship decision will be made by a simple majority vote of the Group's Members.
Votes are due by midnight UTC next Wednesday, July 22nd.
As an optimization, if an absolute majority of the Group's Members votes one way or the other prior
to that time then the decision may be rendered earlier.

Only Members of the JMX Group are eligible to vote on this decision. The current Members are:

Luis-Miguel Alventosa
Jean-François Denise
Joël Féraud
Daniel Fuchs
Shanliang Jiang
Yves Joan
Sandra Lions
Éamonn McManus

Once a decision has been made, the votes will be summarized and report to this list
and also to discuss at openjdk.java.net.

[1] http://mail.openjdk.java.net/pipermail/announce/2009-July/000080.html
--
Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus/

jmx-dev [Fwd: [security-dev 00441]: JMX instrumentation points for security?]

2008-12-05T01:31:54Z

Anyone have suggestions for security things that the JDK could expose through MBeans?

--
Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus

In the last few weeks, I've been chasing an ugly native resource
exhaustion issue, and was able to make use of some of the JMX
instrumentation in areas like NIO.

For example, run a recent jconsole and connect to a JDK7 JVM. Go to the
MBeans tab, then open something like java.nio->BufferPool->Attributes.
You can see the amount of buffer/memory use (double click on the number
to go to a chart view).

Is there some data that would be useful to display here? Most of our
process-related stuff we're dumping is controlled by the System
Properties (javax.net.debug/java.security.debug), and I can't really
think of anything obvious at the system level. Installed/active JCA
security providers maybe?

Anyone have any ideas to propose?

Brad

Re: jmx-dev [PATCH] Fix some compilation problems

2008-09-09T04:45:30Z

Roman,

Thanks for letting us know. I have logged bug 6746196 to track this, and it will be fixed in a forthcoming JDK 7 build.

I agree that it would be good if javac followed the language specification (see bug 6400189). There might be a fear of causing programs that previously compiled to stop doing so, but in that case the right answer is probably a compiler flag, so that we could at least ask for strictness and avoid the conflicts with the Eclipse compiler.

Regards,

Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus/

Roman Kennke wrote:

I think I posted some of this a while ago already, but it seems like
this kind of problems sneaks in every now and then. The JMX code has
some generics code that is not valid Java code but is accepted by javac
anyway (which is a bug in javac, which should be fixed IMO). Other (Java
compatible) compilers (like the Eclipse compiler) reject this code,
which is the correct thing to do. The attached patch fixes the (current)
problems in JMX.

/Roman

jmx-dev [PATCH] Fix some compilation problems

2008-09-09T04:20:55Z

I think I posted some of this a while ago already, but it seems like
this kind of problems sneaks in every now and then. The JMX code has
some generics code that is not valid Java code but is accepted by javac
anyway (which is a bug in javac, which should be fixed IMO). Other (Java
compatible) compilers (like the Eclipse compiler) reject this code,
which is the correct thing to do. The attached patch fixes the (current)
problems in JMX.

/Roman

--
Dipl.-Inform. (FH) Roman Kennke, Software Engineer, http://kennke.org
aicas Allerton Interworks Computer Automated Systems GmbH
Haid-und-Neu-Straße 18 * D-76131 Karlsruhe * Germany
http://www.aicas.com * Tel: +49-721-663 968-48
USt-Id: DE216375633, Handelsregister HRB 109481, AG Karlsruhe
Geschäftsführer: Dr. James J. Hunt

[patch.txt]

diff -r 213263e6dec4 -r 20f1a33a7460 src/share/classes/com/sun/jmx/mbeanserver/DefaultMXBeanMappingFactory.java --- a/src/share/classes/com/sun/jmx/mbeanserver/DefaultMXBeanMappingFactory.java Thu Jul 31 16:49:31 2008 +0200 +++ b/src/share/classes/com/sun/jmx/mbeanserver/DefaultMXBeanMappingFactory.java Fri Aug 01 14:34:36 2008 +0200 @@ -1206,7 +1206,7 @@ // Also remember the set of properties in that constructor // so we can test unambiguity. Set<BitSet> getterIndexSets = newSet(); - for (Constructor constr : annotatedConstrList) { + for (Constructor<?> constr : annotatedConstrList) { String[] propertyNames = constr.getAnnotation(propertyNamesClass).value(); diff -r 00a6f1f4a0e2 src/share/classes/com/sun/jmx/mbeanserver/MBeanIntrospector.java --- a/src/share/classes/com/sun/jmx/mbeanserver/MBeanIntrospector.java Fri Sep 05 16:16:59 2008 +0200 +++ b/src/share/classes/com/sun/jmx/mbeanserver/MBeanIntrospector.java Tue Sep 09 13:15:32 2008 +0200 @@ -623,7 +623,7 @@ } private static MBeanConstructorInfo[] findConstructors(Class<?> c) { - Constructor[] cons = c.getConstructors(); + Constructor<?>[] cons = c.getConstructors(); MBeanConstructorInfo[] mbc = new MBeanConstructorInfo[cons.length]; for (int i = 0; i < cons.length; i++) { String descr = "Public constructor of the MBean";

Re: jmx-dev Need reviewer for 1-liner fix: 6732192: CORE_PKGS.gmk: need to declare javax.management.event in the CORE_PKGS variable

2008-07-31T09:12:22Z

Reviewed OK.

Éamonn McManus · JMX Spec Lead · http://weblogs.java.net/blog/emcmanus/

Daniel Fuchs wrote:

See attached diff...

5108776 has added a new public javax.management.event package.
The javax.management.event package must also be declared
in CORE_PKGS.gmk in order to be included in the javadoc.

cheers,

-- daniel
http://blogs.sun.com/jmxetc

jmx-dev Need reviewer for 1-liner fix: 6732192: CORE_PKGS.gmk: need to declare javax.management.event in the CORE_PKGS variable

2008-07-31T09:03:52Z

See attached diff...

5108776 has added a new public javax.management.event package.
The javax.management.event package must also be declared
in CORE_PKGS.gmk in order to be included in the javadoc.

cheers,

-- daniel
http://blogs.sun.com/jmxetc

diff --git a/make/docs/CORE_PKGS.gmk b/make/docs/CORE_PKGS.gmk
--- a/make/docs/CORE_PKGS.gmk
+++ b/make/docs/CORE_PKGS.gmk
@@ -155,6 +155,7 @@ CORE_PKGS =
javax.lang.model.type \
javax.lang.model.util \
javax.management \
+ javax.management.event \
javax.management.loading \
javax.management.monitor \
javax.management.relation \

jmx-dev Need reviewer for this simple fix: 6402254: Revisit ModelMBean DescriptorSupport implementation of equals and hashCode.

2008-07-28T11:16:37Z

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6402254

see attached diff.

regards,

-- daniel
http://blogs.sun.com/jmxetc

diff --git a/src/share/classes/com/sun/jmx/mbeanserver/Util.java b/src/share/classes/com/sun/jmx/mbeanserver/Util.java
--- a/src/share/classes/com/sun/jmx/mbeanserver/Util.java
+++ b/src/share/classes/com/sun/jmx/mbeanserver/Util.java
@@ -26,6 +26,7 @@ package com.sun.jmx.mbeanserver;
package com.sun.jmx.mbeanserver;

import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collection;
import java.util.Collections;
import java.util.Comparator;
@@ -113,4 +114,32 @@ public class Util {
public static <T> T cast(Object x) {
return (T) x;
}
+
+ /**
+ * Computes a descriptor hashcode from its names and values.
+ * @param names the sorted array of descriptor names.
+ * @param values the array of descriptor values.
+ * @return a hash code value, as described in {@link #hashCode(Descriptor)}
+ */
+ public static int hashCode(String[] names, Object[] values) {
+ int hash = 0;
+ for (int i = 0; i < names.length; i++) {
+ Object v = values[i];
+ int h;
+ if (v == null) {
+ h = 0;
+ } else if (v instanceof Object[]) {
+ h = Arrays.deepHashCode((Object[]) v);
+ } else if (v.getClass().isArray()) {
+ h = Arrays.deepHashCode(new Object[]{v}) - 31;
+ // hashcode of a list containing just v is
+ // v.hashCode() + 31, see List.hashCode()
+ } else {
+ h = v.hashCode();
+ }
+ hash += names[i].toLowerCase().hashCode() ^ h;
+ }
+ return hash;
+ }
+
}
diff --git a/src/share/classes/javax/management/ImmutableDescriptor.java b/src/share/classes/javax/management/ImmutableDescriptor.java
--- a/src/share/classes/javax/management/ImmutableDescriptor.java
+++ b/src/share/classes/javax/management/ImmutableDescriptor.java
@@ -1,5 +1,5 @@
/*
- * Copyright 2004-2006 Sun Microsystems, Inc. All Rights Reserved.
+ * Copyright 2004-2008 Sun Microsystems, Inc. All Rights Reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -25,6 +25,7 @@

package javax.management;

+import com.sun.jmx.mbeanserver.Util;
import java.io.InvalidObjectException;
import java.lang.reflect.Array;
import java.util.Arrays;
@@ -362,6 +363,7 @@ public class ImmutableDescriptor impleme
*/
// Note: this Javadoc is copied from javax.management.Descriptor
// due to 6369229.
+ @Override
public boolean equals(Object o) {
if (o == this)
return true;
@@ -410,29 +412,15 @@ public class ImmutableDescriptor impleme
*/
// Note: this Javadoc is copied from javax.management.Descriptor
// due to 6369229.
+ @Override
public int hashCode() {
if (hashCode == -1) {
- int hash = 0;
- for (int i = 0; i < names.length; i++) {
- Object v = values[i];
- int h;
- if (v == null)
- h = 0;
- else if (v instanceof Object[])
- h = Arrays.deepHashCode((Object[]) v);
- else if (v.getClass().isArray()) {
- h = Arrays.deepHashCode(new Object[] {v}) - 31;
- // hashcode of a list containing just v is
- // v.hashCode() + 31, see List.hashCode()
- } else
- h = v.hashCode();
- hash += names[i].toLowerCase().hashCode() ^ h;
- }
- hashCode = hash;
+ hashCode = Util.hashCode(names, values);
}
return hashCode;
}

+ @Override
public String toString() {
StringBuilder sb = new StringBuilder("{");
for (int i = 0; i < names.length; i++) {
@@ -479,6 +467,7 @@ public class ImmutableDescriptor impleme
* If the descriptor construction fails for any reason, this exception will
* be thrown.
*/
+ @Override
public Descriptor clone() {
return this;
}
diff --git a/src/share/classes/javax/management/modelmbean/DescriptorSupport.java b/src/share/classes/javax/management/modelmbean/DescriptorSupport.java
--- a/src/share/classes/javax/management/modelmbean/DescriptorSupport.java
+++ b/src/share/classes/javax/management/modelmbean/DescriptorSupport.java
@@ -33,6 +33,7 @@ import static com.sun.jmx.defaults.JmxPr
import static com.sun.jmx.defaults.JmxProperties.MODELMBEAN_LOGGER;
import static com.sun.jmx.mbeanserver.Util.cast;
import com.sun.jmx.mbeanserver.GetPropertyAction;
+import com.sun.jmx.mbeanserver.Util;

import java.io.IOException;
import java.io.ObjectInputStream;
@@ -774,6 +775,7 @@ public class DescriptorSupport
* fails for any reason, this exception will be thrown.
*/

+ @Override
public synchronized Object clone() throws RuntimeOperationsException {
if (MODELMBEAN_LOGGER.isLoggable(Level.FINEST)) {
MODELMBEAN_LOGGER.logp(Level.FINEST,
@@ -814,13 +816,16 @@ public class DescriptorSupport
* otherwise.
*
*/
- // XXXX TODO: This is not very efficient!
// Note: this Javadoc is copied from javax.management.Descriptor
// due to 6369229.
+ @Override
public synchronized boolean equals(Object o) {
- if (o == this)
+ if (o == this)
return true;
-
+ if (! (o instanceof Descriptor))
+ return false;
+ if (o instanceof ImmutableDescriptor)
+ return o.equals(this);
return new ImmutableDescriptor(descriptorMap).equals(o);
}

@@ -844,11 +849,16 @@ public class DescriptorSupport
* @return A hash code value for this object.
*
*/
- // XXXX TODO: This is not very efficient!
// Note: this Javadoc is copied from javax.management.Descriptor
// due to 6369229.
+ @Override
public synchronized int hashCode() {
- return new ImmutableDescriptor(descriptorMap).hashCode();
+ final int size = descriptorMap.size();
+ // descriptorMap is sorted with a comparator that ignore case.
+ //
+ return Util.hashCode(
+ descriptorMap.keySet().toArray(new String[size]),
+ descriptorMap.values().toArray(new Object[size]));
}

/**
@@ -1278,6 +1288,7 @@ public class DescriptorSupport
* field Names or field Values. If the descriptor string fails
* for any reason, this exception will be thrown.
*/
+ @Override
public synchronized String toString() {
if (MODELMBEAN_LOGGER.isLoggable(Level.FINEST)) {
MODELMBEAN_LOGGER.logp(Level.FINEST,

Re: jmx-dev Review request: 6323980: Annotations to simplify MBean development

2008-06-19T08:34:55Z

Hi Eamonn,

Sorry for the late review. Here is my feedback:

com/sun/jmx/mbeanserver/Introspector.java

getStandardMBeanInterface()
can now also return a class (not necessarily an interface).
Maybe the method javadoc could say so.

addDescriptorFieldsToMap(), addAnnotationFieldsToMap():
I think I understand the logic there, but if I ever have to
fix something in this code I'll be very grateful to find a comment
with an example of annotated class and an explanation of what each
of these methods looks for in the annotated class.

test/javax/management/Introspector/AnnotatedMBeanTest.java

This test could also check that Impact=UNKNOWN by default, and
add an @ManagedOperation method where Impact is set to
something else.

-- daniel
http://blogs.sun.com/jmxetc

Eamonn McManus wrote:

> I'm looking for reviewers for this fairly substantial change to the JMX
> API and implementation in JDK7. I need at least one reviewer with commit
> rights to openjdk. Any other reviewers will be acknowledged in the
> changeset message.
>
> The context:
> http://weblogs.java.net/blog/emcmanus/archive/2007/08/defining_mbeans.html
>
> The specification changes:
> http://weblogs.java.net/blog/emcmanus/archive/6323980-specdiff.html
>
> The code changes:
> http://weblogs.java.net/blog/emcmanus/archive/6323980-webrev.zip
>
> Thanks,
>

Re: jmx-dev Review request: 6323980: Annotations to simplify MBean development

2008-06-10T01:46:48Z

JF,

Excellent point! Although we might imagine that someone would want to
reuse the @Description annotation for another purpose where FIELD would
be valid, I don't think they would gain much doing that versus creating
their own @Description with the semantics they want. So we should remove
FIELD from the @Target, at least until we do use it for something.

Thanks,

Éamonn McManus JMX Spec Lead http://weblogs.java.net/blog/emcmanus/

Jean-Francois Denise wrote:

> Hello Eamonn,
> I mainly focus on new Annotations and impact on public classes. I
> noticed that Description Annotation targets FIELD. I was not able to
> find a case where the FIELD target is needed. But perhaps that I
> missed something there.
>
> JF.
>
>
> Eamonn McManus wrote:
>> I'm looking for reviewers for this fairly substantial change to the
>> JMX API and implementation in JDK7. I need at least one reviewer with
>> commit rights to openjdk. Any other reviewers will be acknowledged in
>> the changeset message.
>>
>> The context:
>> http://weblogs.java.net/blog/emcmanus/archive/2007/08/defining_mbeans.html
>>
>>
>> The specification changes:
>> http://weblogs.java.net/blog/emcmanus/archive/6323980-specdiff.html
>>
>> The code changes:
>> http://weblogs.java.net/blog/emcmanus/archive/6323980-webrev.zip
>>
>> Thanks,
>>
>
>

Re: jmx-dev Review request: 6323980: Annotations to simplify MBean development

2008-06-10T00:11:30Z

Hello Eamonn,
I mainly focus on new Annotations and impact on public classes. I
noticed that Description Annotation targets FIELD. I was not able to
find a case where the FIELD target is needed. But perhaps that I missed
something there.

JF.

Eamonn McManus wrote:

> I'm looking for reviewers for this fairly substantial change to the
> JMX API and implementation in JDK7. I need at least one reviewer with
> commit rights to openjdk. Any other reviewers will be acknowledged in
> the changeset message.
>
> The context:
> http://weblogs.java.net/blog/emcmanus/archive/2007/08/defining_mbeans.html
>
>
> The specification changes:
> http://weblogs.java.net/blog/emcmanus/archive/6323980-specdiff.html
>
> The code changes:
> http://weblogs.java.net/blog/emcmanus/archive/6323980-webrev.zip
>
> Thanks,
>

--
Jean-Francois Denise JMX team
Interested in netbeans ==> http://blogs.sun.com/roller/page/jmxnetbeans

jmx-dev Review request: 6323980: Annotations to simplify MBean development

2008-06-09T08:48:43Z

I'm looking for reviewers for this fairly substantial change to the JMX
API and implementation in JDK7. I need at least one reviewer with commit
rights to openjdk. Any other reviewers will be acknowledged in the
changeset message.

The context:
http://weblogs.java.net/blog/emcmanus/archive/2007/08/defining_mbeans.html

The specification changes:
http://weblogs.java.net/blog/emcmanus/archive/6323980-specdiff.html

The code changes:
http://weblogs.java.net/blog/emcmanus/archive/6323980-webrev.zip

Thanks,

--
Éamonn McManus JMX Spec Lead http://weblogs.java.net/blog/emcmanus/

Re: jmx-dev Request for review: CR 6711106: REGRESSION: Bad usage of SnapshotMBeanServerConnection in MBeans tab and JConsole plugins.

2008-06-05T08:35:56Z

Reviewed OK.

Luis-Miguel Alventosa wrote:
> FYI
>
> Looking for a reviewer in order to commit the changes into OpenJDK 7.
>
> Thanks,
> Luis

--
Jean-Francois Denise JMX team
Interested in netbeans ==> http://blogs.sun.com/roller/page/jmxnetbeans

jmx-dev Request for review: CR 6711106: REGRESSION: Bad usage of SnapshotMBeanServerConnection in MBeans tab and JConsole plugins.

2008-06-05T08:00:01Z

FYI

Looking for a reviewer in order to commit the changes into OpenJDK 7.

Thanks,
Luis

--- old/src/share/classes/sun/tools/jconsole/MBeansTab.java Thu Jun 5 16:57:36 2008
+++ new/src/share/classes/sun/tools/jconsole/MBeansTab.java Thu Jun 5 16:57:35 2008
@@ -37,6 +37,7 @@
import javax.swing.*;
import javax.swing.event.*;
import javax.swing.tree.*;
+import sun.tools.jconsole.ProxyClient.SnapshotMBeanServerConnection;
import sun.tools.jconsole.inspector.*;

import com.sun.tools.jconsole.JConsoleContext;
@@ -154,6 +155,10 @@
return vmPanel.getProxyClient().getMBeanServerConnection();
}

+ public SnapshotMBeanServerConnection getSnapshotMBeanServerConnection() {
+ return vmPanel.getProxyClient().getSnapshotMBeanServerConnection();
+ }
+
@Override
public void update() {
// Ping the connection to see if it is still alive. At
--- old/src/share/classes/sun/tools/jconsole/ProxyClient.java Thu Jun 5 16:57:38 2008
+++ new/src/share/classes/sun/tools/jconsole/ProxyClient.java Thu Jun 5 16:57:38 2008
@@ -28,7 +28,6 @@
import com.sun.management.HotSpotDiagnosticMXBean;
import com.sun.tools.jconsole.JConsoleContext;
import com.sun.tools.jconsole.JConsoleContext.ConnectionState;
-import java.awt.Component;
import java.beans.PropertyChangeListener;
import java.beans.PropertyChangeEvent;
import java.io.IOException;
@@ -78,6 +77,7 @@
private String advancedUrl = null;

private JMXServiceURL jmxUrl = null;
+ private MBeanServerConnection mbsc = null;
private SnapshotMBeanServerConnection server = null;
private JMXConnector jmxc = null;
private RMIServer stub = null;
@@ -103,7 +103,6 @@

private List<MemoryPoolProxy> memoryPoolProxies = null;
private List<GarbageCollectorMXBean> garbageCollectorMBeans = null;
- private String detectDeadlocksOperation = null;

final static private String HOTSPOT_DIAGNOSTIC_MXBEAN_NAME =
"com.sun.management:type=HotSpotDiagnostic";
@@ -326,8 +325,8 @@
if (jmxUrl == null && "localhost".equals(hostName) && port == 0) {
// Monitor self
this.jmxc = null;
- this.server = Snapshot.newSnapshot(
- ManagementFactory.getPlatformMBeanServer());
+ this.mbsc = ManagementFactory.getPlatformMBeanServer();
+ this.server = Snapshot.newSnapshot(mbsc);
} else {
// Monitor another process
if (lvm != null) {
@@ -369,7 +368,8 @@
this.jmxc = JMXConnectorFactory.connect(jmxUrl, env);
}
}
- this.server = Snapshot.newSnapshot(jmxc.getMBeanServerConnection());
+ this.mbsc = jmxc.getMBeanServerConnection();
+ this.server = Snapshot.newSnapshot(mbsc);
}
this.isDead = false;

@@ -518,7 +518,11 @@
}
}

- public MBeanServerConnection getMBeanServerConnection() {
+ public MBeanServerConnection getMBeanServerConnection() {
+ return mbsc;
+ }
+
+ public SnapshotMBeanServerConnection getSnapshotMBeanServerConnection() {
return server;
}

--- old/src/share/classes/sun/tools/jconsole/inspector/XMBean.java Thu Jun 5 16:57:40 2008
+++ new/src/share/classes/sun/tools/jconsole/inspector/XMBean.java Thu Jun 5 16:57:40 2008
@@ -30,6 +30,7 @@
import javax.swing.Icon;
import sun.tools.jconsole.JConsole;
import sun.tools.jconsole.MBeansTab;
+import sun.tools.jconsole.ProxyClient.SnapshotMBeanServerConnection;

public class XMBean {

@@ -60,6 +61,10 @@
return mbeansTab.getMBeanServerConnection();
}

+ SnapshotMBeanServerConnection getSnapshotMBeanServerConnection() {
+ return mbeansTab.getSnapshotMBeanServerConnection();
+ }
+
public Boolean isBroadcaster() {
synchronized (broadcasterLock) {
if (broadcaster == null) {
@@ -103,7 +108,7 @@
public Object getAttribute(String attributeName)
throws AttributeNotFoundException, InstanceNotFoundException,
MBeanException, ReflectionException, IOException {
- return getMBeanServerConnection().getAttribute(
+ return getSnapshotMBeanServerConnection().getAttribute(
getObjectName(), attributeName);
}

@@ -110,7 +115,7 @@
public AttributeList getAttributes(String attributeNames[])
throws AttributeNotFoundException, InstanceNotFoundException,
MBeanException, ReflectionException, IOException {
- return getMBeanServerConnection().getAttributes(
+ return getSnapshotMBeanServerConnection().getAttributes(
getObjectName(), attributeNames);
}

--- old/src/share/classes/sun/tools/jconsole/inspector/XMBeanAttributes.java Thu Jun 5 16:57:42 2008
+++ new/src/share/classes/sun/tools/jconsole/inspector/XMBeanAttributes.java Thu Jun 5 16:57:42 2008
@@ -349,7 +349,14 @@

try {
list = mbean.getAttributes(attributesInfo);
- }catch(Exception e) {
+ } catch (Exception e) {
+ if (JConsole.isDebug()) {
+ System.err.println("Error calling getAttributes() on MBean \"" +
+ mbean.getObjectName() + "\". JConsole will " +
+ "try to get them individually calling " +
+ "getAttribute() instead. Exception:");
+ e.printStackTrace(System.err);
+ }
list = new AttributeList();
//Can't load all attributes, do it one after each other.
for(int i = 0; i < attributesInfo.length; i++) {
@@ -357,7 +364,7 @@
try {
name = attributesInfo[i].getName();
Object value =
- mbean.getAttribute(name);
+ mbean.getMBeanServerConnection().getAttribute(mbean.getObjectName(), name);
list.add(new Attribute(name, value));
}catch(Exception ex) {
if(attributesInfo[i].isReadable()) {
@@ -397,8 +404,8 @@
// went wrong.
try {
Object v =
- mbean.getAttribute(attributeInfo.
- getName());
+ mbean.getMBeanServerConnection().getAttribute(
+ mbean.getObjectName(), attributeInfo.getName());
//What happens if now it is ok?
// Be pragmatic, add it to readable...
attributes.put(attributeInfo.getName(),
@@ -528,10 +535,8 @@
}

public void refreshAttributes() {
- MBeanServerConnection mbsc = mbeansTab.getMBeanServerConnection();
- if (mbsc instanceof SnapshotMBeanServerConnection) {
- ((SnapshotMBeanServerConnection) mbsc).flush();
- }
+ SnapshotMBeanServerConnection mbsc = mbeansTab.getSnapshotMBeanServerConnection();
+ mbsc.flush();
stopCellEditing();
loadAttributes(mbean, mbeanInfo);
}

Re: jmx-dev Review requested: 6592586: RequiredModelMBean prints a WARNING message when calling getAttributes() for a non-existing attr

2008-05-30T05:14:44Z

Eamonn McManus wrote:
> Reviewed OK. Level.FINE might be more appropriate than Level.FINER for
> exceptions from writeToLog.

OK, good feedback. I will make the modifications.

>
> Éamonn McManus JMX Spec Lead http://weblogs.java.net/blog/emcmanus/
>
>
>
> Daniel Fuchs wrote:
>> Also fixed several occurrences of the WARNING level....
>>
>> -- daniel
>> http://blogs.sun.com/jmxetc
>>
>> The attachment(s) "" has been removed from this email because it
>> contained a type of attachment known to carry viruses and other
>> harmful software. The attachment has been removed for your protection,
>> as well as that of Sun's network.
>> If you know the sender and were expecting the attachment, please
>> contact the sender and have them re-send the file using the .zip
>> format. If the sender is unknown or if the email looks unusual or is
>> unexpected, please delete it now.
>> Additional information: http://antivirus.central.sun.com/
>

Re: jmx-dev Review requested: 6592586: RequiredModelMBean prints a WARNING message when calling getAttributes() for a non-existing attr

2008-05-30T04:02:05Z

Reviewed OK. Level.FINE might be more appropriate than Level.FINER for
exceptions from writeToLog.

Éamonn McManus JMX Spec Lead http://weblogs.java.net/blog/emcmanus/

Daniel Fuchs wrote:

> Also fixed several occurrences of the WARNING level....
>
> -- daniel
> http://blogs.sun.com/jmxetc
>
> The attachment(s) "" has been removed from this email because it
> contained a type of attachment known to carry viruses and other
> harmful software. The attachment has been removed for your protection,
> as well as that of Sun's network.
> If you know the sender and were expecting the attachment, please
> contact the sender and have them re-send the file using the .zip
> format. If the sender is unknown or if the email looks unusual or is
> unexpected, please delete it now.
> Additional information: http://antivirus.central.sun.com/

jmx-dev Review requested: 6592586: RequiredModelMBean prints a WARNING message when calling getAttributes() for a non-existing attr

2008-05-29T08:49:40Z

Also fixed several occurrences of the WARNING level....

-- daniel
http://blogs.sun.com/jmxetc

--- /jmgt/user/dfuchs/openjdk/jdk/webrev/raw_files/old/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java Thu May 29 17:32:30 2008
+++ /jmgt/user/dfuchs/openjdk/jdk/webrev/raw_files/new/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java Thu May 29 17:32:30 2008
@@ -1696,8 +1696,8 @@
} catch (Exception e) {
// eat exceptions because interface doesn't have an
// exception on it
- if (MODELMBEAN_LOGGER.isLoggable(Level.WARNING)) {
- MODELMBEAN_LOGGER.logp(Level.WARNING,
+ if (MODELMBEAN_LOGGER.isLoggable(Level.FINER)) {
+ MODELMBEAN_LOGGER.logp(Level.FINER,
RequiredModelMBean.class.getName(),
"getAttributes(String[])",
"Failed to get \"" + attrNames[i] + "\": ", e);
@@ -1857,8 +1857,8 @@
attrValue.getClass().getName() +
" received.");
} catch (ClassNotFoundException x) {
- if (MODELMBEAN_LOGGER.isLoggable(Level.WARNING)) {
- MODELMBEAN_LOGGER.logp(Level.WARNING,
+ if (MODELMBEAN_LOGGER.isLoggable(Level.FINER)) {
+ MODELMBEAN_LOGGER.logp(Level.FINER,
RequiredModelMBean.class.getName(),
"setAttribute(Attribute)","Class " +
attrType + " for attribute "
@@ -2224,8 +2224,8 @@
ntfyObj.getMessage() + " Severity = " +
(String)ntfyDesc.getFieldValue("severity"));
} catch (Exception e) {
- if (MODELMBEAN_LOGGER.isLoggable(Level.WARNING)) {
- MODELMBEAN_LOGGER.logp(Level.WARNING,
+ if (MODELMBEAN_LOGGER.isLoggable(Level.FINER)) {
+ MODELMBEAN_LOGGER.logp(Level.FINER,
RequiredModelMBean.class.getName(),
"sendNotification(Notification)",
"Failed to log " +
@@ -2618,8 +2618,8 @@
" Old value = " + oldv +
" New value = " + newv);
} catch (Exception e) {
- if (MODELMBEAN_LOGGER.isLoggable(Level.WARNING)) {
- MODELMBEAN_LOGGER.logp(Level.WARNING,
+ if (MODELMBEAN_LOGGER.isLoggable(Level.FINER)) {
+ MODELMBEAN_LOGGER.logp(Level.FINER,
RequiredModelMBean.class.getName(),mth,
"Failed to log " + ntfyObj.getType() +
" notification: ", e);
@@ -2644,8 +2644,8 @@
" Old value = " + oldv +
" New value = " + newv);
} catch (Exception e) {
- if (MODELMBEAN_LOGGER.isLoggable(Level.WARNING)) {
- MODELMBEAN_LOGGER.logp(Level.WARNING,
+ if (MODELMBEAN_LOGGER.isLoggable(Level.FINER)) {
+ MODELMBEAN_LOGGER.logp(Level.FINER,
RequiredModelMBean.class.getName(),mth,
"Failed to log " + ntfyObj.getType() +
" notification: ", e);

webrev.zip (175K) Download Attachment

Re: jmx-dev hg: jdk7/tl/jdk: 6701459: Synchronization bug pattern found in javax.management.relation.RelationService

2008-05-15T02:59:57Z

Rob,

You write:

In the change sets some variable postfixes were renamed
Nbr  -> No

Example:
Long seqNbr = getNotificationSequenceNumber();
+ Long seqNo = atomicSeqNo.incrementAndGet();


The abbreviation No could also could mean:
 No as in 'Yes'/'No' ,
 No as in 'Notation',
 besides 'Number'.
Generally avoiding unclear naming seems desirable.
Comments ?

First of all, I'm very happy to see someone paying this much attention to my changes!

You are probably right: I think seqNum is a better abbreviation than seqNo. On the other hand, seqNo is the abbreviation used in the few other places in the JMX source where there is a local variable or parameter that is a sequence number. Google shows 313,000 hits for "seqnum" and 5,480,000 for "seqno" so we might conclude that both are fairly standard abbreviations with "seqno" being commoner. "seqnbr" comes in a poor third at 10,100 so I think I was justified in renaming it in any case. (Google Code Search shows about equal numbers for seqno and seqnum.)

We could do a sweep over the JMX sources to rename the various seqNo (and paramNo) variables to seqNum (paramNum). But there's a tension here between slightly improved readability and the noise that would be generated by such a change. However, I'll bear your suggestion in mind for future changes, and this is also a reminder for me that I should have sent the diffs to the public jmx-dev@... list for review.

Thanks,

-- 
Éamonn McManus   JMX Spec Lead   http://weblogs.java.net/blog/emcmanus/

Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

2008-05-07T05:43:09Z

Thanks Daniel

It certainly has helped me. Clarified many things. :-)

Thanks for your clarifications.

Regards
-Nitin

Daniel Fuchs ---05/06/2008 09:43:15 PM---Hi Nitin, all,

Daniel Fuchs <Daniel.Fuchs@...>

Sent by: Daniel.Fuchs@...

05/06/2008 09:39 PM

Please respond to
Daniel.Fuchs@...

To	Nitin Agrawal/San Francisco/Contr/IBM@IBMUS
cc	jmx-dev <jmx-dev@...>, Ken Ellinwood/Los Angeles/IBM@IBMUS
Subject	Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

Hi Nitin, all, Nitin Agrawal wrote: > We are using javax.management.snmp APIs. The javax.management.snmp package is an old deprecated package name. The SNMP APIs where candidate for inclusion in JMX, but they never made it to the final JMX specification. Since then the javax.management.snmp packages have been renamed into com.sun.management.snmp, which is provided by JDMK 5.1 or OpenDMK. > In our Management application, > we want to parse the OID data (from a SNMP GET request or a trap), where > in the OID type is an IPAddress and the value can be either IPv4 or IPv6 > address. To represent this we are using > "javax.management.snmp.SnmpIpAddress" class. This supports IPv4 > addresses only. SnmpIpAddress corresponds to the IpAddress syntax defined in RFC 1155.ftp://ftp.rfc-editor.org/in-notes/rfc1155.txtIt is a 4 byte Octet String and can only be used to transport IPv4 address. An object using the IpAddress syntax cannot be used to hold an IPv6 address. > I also checked on the Open DMK site: >https://opendmk.dev.java.net/download/index.html#opendmk-1.0-b02 and > downloaded - opendmk-1.0-b02-src-dual-01-Oct-2007_19-17-46.zip > > The class - > OpenDMK-src\src\snmp_manager\com\sun\management\snmp\SnmpIpAddress.java > (Attached here: /(See attached file: SnmpIpAddress.java)/ ) does not > have IPv6 support. It can't. This is not a limitation of the API - it's a limitation of of the SNMP definitions in your MIB. > Are you aware of any implementation of such APIs which has got IPv6 > support ? You need to upgrade your MIB and use an object based on one of the textual conventions defined in the INET-ADDRESS-MIB (RFC 4001)ftp://ftp.rfc-editor.org/in-notes/rfc4001.txt> If I see the J2SE JMX API specs @ >http://java.sun.com/j2se/1.5.0/docs/guide/jmx/spec.html, > "javax.management.snmp" is not there. Looks like formal support for SNMP > APIs has been dropped by sun. Not at all. As explained these APIs are exported by Java DMK 5.1 and OpenDMK. They re not part of the Java platform. See here for more details:http://blogs.sun.com/jmxetc/entry/what_is_the_jvm_snmpHope this helps, -- danielhttp://blogs.sun.com/jmxetc> > Regards > -Nitin > > > > > Inactive hide details for Daniel Fuchs ---05/05/2008 10:12:58 > PM---Hi,Daniel Fuchs ---05/05/2008 10:12:58 PM---Hi, > > *Daniel Fuchs <Daniel.Fuchs@...>* > Sent by: Daniel.Fuchs@... > > 05/05/2008 09:57 PM > Please respond to > Daniel.Fuchs@... > > > > To > > Nitin Agrawal/San Francisco/Contr/IBM@IBMUS > > cc > > jmx-dev <jmx-dev@...>, Ken Ellinwood/Los Angeles/IBM@IBMUS > > Subject > > Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs > > > > > Hi, > > Yes, IPv6 is supported. > > You can specify IPv6 addresses in the ACL files, and you can > specify a specific interface to which to bind to (if you want > to bind to an interface which is not the default) using > the com.sun.management.snmp.interface property. > > for instance, you can define: > com.sun.management.snmp.interface=::1 > > See <jdk-install>/jre/lib/management/snmp.acl.template > and <jdk-install>/jre/lib/management/management.properties > > Hope this helps, > > -- daniel >http://blogs.sun.com/jmxetc> > Nitin Agrawal wrote: > > Hi > > > > Apologies if it is a repeat question, but I could not find an answer to > > it onhttp://openjdk.java.net/groups/jmx/ and in the mailing list > archives. > > > > *Is IPv6 supported* in JMX SNMP APIs (javax.management and sub-packages) > > with JDK 1.5? I have searched on Sun's site and googled it, but could > > not get any lead on it. > > > > In our application, we are using these APIs which are only IPv4 > > compliant. We are adding IPv6 support and hence this query. > > > > If there is already IPv6 support available for 1.5, please point me to > > it. If it is not, then:- > > - Please let me know in case if it supported in JDK 1.6 or even here > > there is no support for IPv6. > > > > If there is not support anywhere, we need to go with DIY approach and > > (re-)invent the wheel. > > > > Will appreciate any thoughts/pointers on this. > > > > Regards > > -Nitin Agrawal > > > >

pic02660.gif (1K) Download Attachment

Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

2008-05-06T09:09:26Z

>
> Regards
> -Nitin
>
>
>
>
> Inactive hide details for Daniel Fuchs ---05/05/2008 10:12:58
> PM---Hi,Daniel Fuchs ---05/05/2008 10:12:58 PM---Hi,
>
> *Daniel Fuchs <Daniel.Fuchs@...>*
> Sent by: Daniel.Fuchs@...
>
> 05/05/2008 09:57 PM
> Please respond to
> Daniel.Fuchs@...
>
>
>
> To
>
> Nitin Agrawal/San Francisco/Contr/IBM@IBMUS
>
> cc
>
> jmx-dev <jmx-dev@...>, Ken Ellinwood/Los Angeles/IBM@IBMUS
>
> Subject
>
> Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs
>
>
>
>
> Hi,
>
> Yes, IPv6 is supported.
>
> You can specify IPv6 addresses in the ACL files, and you can
> specify a specific interface to which to bind to (if you want
> to bind to an interface which is not the default) using
> the com.sun.management.snmp.interface property.
>
> for instance, you can define:
> com.sun.management.snmp.interface=::1
>
> See <jdk-install>/jre/lib/management/snmp.acl.template
> and <jdk-install>/jre/lib/management/management.properties
>
> Hope this helps,
>
> -- daniel
> http://blogs.sun.com/jmxetc
>
> Nitin Agrawal wrote:
> > Hi
> >
> > Apologies if it is a repeat question, but I could not find an answer to
> > it on http://openjdk.java.net/groups/jmx/ and in the mailing list
> archives.
> >
> > *Is IPv6 supported* in JMX SNMP APIs (javax.management and sub-packages)
> > with JDK 1.5? I have searched on Sun's site and googled it, but could
> > not get any lead on it.
> >
> > In our application, we are using these APIs which are only IPv4
> > compliant. We are adding IPv6 support and hence this query.
> >
> > If there is already IPv6 support available for 1.5, please point me to
> > it. If it is not, then:-
> > - Please let me know in case if it supported in JDK 1.6 or even here
> > there is no support for IPv6.
> >
> > If there is not support anywhere, we need to go with DIY approach and
> > (re-)invent the wheel.
> >
> > Will appreciate any thoughts/pointers on this.
> >
> > Regards
> > -Nitin Agrawal
> >
>
>

Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

2008-05-06T07:37:48Z

Thanks Daniel for your response.

It overall does not address my concerns/query. I guess I have not given sufficient data to you in my query.

We are using javax.management.snmp APIs. In our Management application, we want to parse the OID data (from a SNMP GET request or a trap), where in the OID type is an IPAddress and the value can be either IPv4 or IPv6 address. To represent this we are using "javax.management.snmp.SnmpIpAddress" class. This supports IPv4 addresses only.

I also checked on the Open DMK site: https://opendmk.dev.java.net/download/index.html#opendmk-1.0-b02 and downloaded - opendmk-1.0-b02-src-dual-01-Oct-2007_19-17-46.zip

The class - OpenDMK-src\src\snmp_manager\com\sun\management\snmp\SnmpIpAddress.java (Attached here: (See attached file: SnmpIpAddress.java) ) does not have IPv6 support.

Are you aware of any implementation of such APIs which has got IPv6 support ?

If I see the J2SE JMX API specs @ http://java.sun.com/j2se/1.5.0/docs/guide/jmx/spec.html, "javax.management.snmp" is not there. Looks like formal support for SNMP APIs has been dropped by sun.

Regards
-Nitin

Daniel Fuchs ---05/05/2008 10:12:58 PM---Hi,

Daniel Fuchs <Daniel.Fuchs@...>

Sent by: Daniel.Fuchs@...

05/05/2008 09:57 PM

Please respond to
Daniel.Fuchs@...

To	Nitin Agrawal/San Francisco/Contr/IBM@IBMUS
cc	jmx-dev <jmx-dev@...>, Ken Ellinwood/Los Angeles/IBM@IBMUS
Subject	Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

Hi, Yes, IPv6 is supported. You can specify IPv6 addresses in the ACL files, and you can specify a specific interface to which to bind to (if you want to bind to an interface which is not the default) using the com.sun.management.snmp.interface property. for instance, you can define: com.sun.management.snmp.interface=::1 See <jdk-install>/jre/lib/management/snmp.acl.template and <jdk-install>/jre/lib/management/management.properties Hope this helps, -- danielhttp://blogs.sun.com/jmxetcNitin Agrawal wrote: > Hi > > Apologies if it is a repeat question, but I could not find an answer to > it onhttp://openjdk.java.net/groups/jmx/ and in the mailing list archives. > > *Is IPv6 supported* in JMX SNMP APIs (javax.management and sub-packages) > with JDK 1.5? I have searched on Sun's site and googled it, but could > not get any lead on it. > > In our application, we are using these APIs which are only IPv4 > compliant. We are adding IPv6 support and hence this query. > > If there is already IPv6 support available for 1.5, please point me to > it. If it is not, then:- > - Please let me know in case if it supported in JDK 1.6 or even here > there is no support for IPv6. > > If there is not support anywhere, we need to go with DIY approach and > (re-)invent the wheel. > > Will appreciate any thoughts/pointers on this. > > Regards > -Nitin Agrawal >

pic05136.gif (1K) Download Attachment
SnmpIpAddress.java (10K) Download Attachment

Re: jmx-dev Query: IPv6 Support in JMX SNMP APIs

2008-05-05T09:27:28Z

Hi,

Yes, IPv6 is supported.

You can specify IPv6 addresses in the ACL files, and you can
specify a specific interface to which to bind to (if you want
to bind to an interface which is not the default) using
the com.sun.management.snmp.interface property.

for instance, you can define:
com.sun.management.snmp.interface=::1

See <jdk-install>/jre/lib/management/snmp.acl.template
and <jdk-install>/jre/lib/management/management.properties

Hope this helps,

-- daniel
http://blogs.sun.com/jmxetc

Nitin Agrawal wrote:

> Hi
>
> Apologies if it is a repeat question, but I could not find an answer to
> it on http://openjdk.java.net/groups/jmx/ and in the mailing list archives.
>
> *Is IPv6 supported* in JMX SNMP APIs (javax.management and sub-packages)
> with JDK 1.5? I have searched on Sun's site and googled it, but could
> not get any lead on it.
>
> In our application, we are using these APIs which are only IPv4
> compliant. We are adding IPv6 support and hence this query.
>
> If there is already IPv6 support available for 1.5, please point me to
> it. If it is not, then:-
> - Please let me know in case if it supported in JDK 1.6 or even here
> there is no support for IPv6.
>
> If there is not support anywhere, we need to go with DIY approach and
> (re-)invent the wheel.
>
> Will appreciate any thoughts/pointers on this.
>
> Regards
> -Nitin Agrawal
>

jmx-dev Query: IPv6 Support in JMX SNMP APIs

2008-05-05T07:12:30Z

Hi

Apologies if it is a repeat question, but I could not find an answer to it on http://openjdk.java.net/groups/jmx/ and in the mailing list archives.

Is IPv6 supported in JMX SNMP APIs (javax.management and sub-packages) with JDK 1.5? I have searched on Sun's site and googled it, but could not get any lead on it.

In our application, we are using these APIs which are only IPv4 compliant. We are adding IPv6 support and hence this query.

If there is already IPv6 support available for 1.5, please point me to it. If it is not, then:-
- Please let me know in case if it supported in JDK 1.6 or even here there is no support for IPv6.

If there is not support anywhere, we need to go with DIY approach and (re-)invent the wheel.

Will appreciate any thoughts/pointers on this.

Regards
-Nitin Agrawal

jmx-dev Vote results: JMX Group sponsorship of the Java VisualVM , Troubleshooting Tool project PASSES

2008-04-28T08:32:35Z

Hi,

The proposal for the JMX Group to sponsor the creation of the
Java VisualVM, Troubleshooting Tool project [1] succeeds with
8 votes for and 0 votes against.

In answer to the question "Should the JMX Group sponsor this Project?"
the individual votes are:

Daniel Fuchs yes
Eamonn McManus yes
Jean-Francois Denise yes
Joel Feraud yes
Luis-Miguel Alventosa yes
Sandra Lions yes
Shanliang Jiang yes
Yves Joan yes

[1] is:
http://mail.openjdk.java.net/pipermail/jmx-dev/2008-April/000019.html

best regards,

-- daniel
http://blogs.sun.com/jmxetc

Re: jmx-dev CFV: Java VisualVM ,Troubleshooting Tool Project

2008-04-28T04:33:19Z

Vote: yes

Daniel Fuchs wrote:

> Hi,
>
> Luis Miguel Alventosa has proposed the "Java VisualVM
> Troubleshooting Tool" Project [1].
>
> Should the JMX Group sponsor this Project?
>
> Please cast your vote by replying to this message with either
> Vote: yes
> or
> Vote: no
> as the first line of the message body.
>
> You may indicate the reason for your decision, if you wish, on
> subsequent lines.
> This is optional and not required.
>
> Votes are due by midnight UTC Friday, 2 May 2008, after which time
> I'll tally the votes and post a summary to this list and to discuss
> at openjdk.java.net.
>
> Only Members of the JMX Group are eligible to vote on this decision [2].
>
> The current Members are:
>
> Daniel Fuchs
> Eamonn McManus
> Jean-Francois Denise
> Joel Feraud
> Luis-Miguel Alventosa
> Sandra Lions
> Shanliang Jiang
> Yves Joan
>
>
> [1] is:
> http://mail.openjdk.java.net/pipermail/announce/2008-April/000044.html
> [2] http://blogs.sun.com/mr/entry/cosmology
>
> -- daniel
> http://blogs.sun.com/jmxetc