[jboss-jira] [JBoss JIRA] Commented: (JBMESSAGING-1247) Remove requirement for JMX to use JGroups
Jeff Mesnil (JIRA)
jira-events at lists.jboss.org
Fri Mar 14 05:09:58 EDT 2008
[ http://jira.jboss.com/jira/browse/JBMESSAGING-1247?page=comments#action_12402857 ]
Jeff Mesnil commented on JBMESSAGING-1247:
------------------------------------------
as part of this task, we should also clean up MessagingPostOfficeService API.
ChannelPartitionName and GroupName serve the same purpose: identify the JGroups channels used by JBoss Messaging.
- ChannelPartitionName is used by AS 5 config in the multiplexer case
- GroupName is used for all the other use case
There should be only one property to set to uniquely identify the channels regardless of the use cases
> Remove requirement for JMX to use JGroups
> -----------------------------------------
>
> Key: JBMESSAGING-1247
> URL: http://jira.jboss.com/jira/browse/JBMESSAGING-1247
> Project: JBoss Messaging
> Issue Type: Task
> Affects Versions: 1.4.1.beta1
> Reporter: Brian Stansberry
> Assigned To: Jeff Mesnil
> Fix For: 1.4.1.beta2
>
>
> The current JGroups integration uses JMX to get ahold of a Channel:
> public Channel createControlChannel() throws Exception
> {
> return (Channel) server.invoke(this.channelFactory, MUX_OPERATION,
> new Object[]{controlStack, uniqueID + "-CTRL", Boolean.TRUE, uniqueID}, MUX_SIGNATURE);
> }
> The channelFactory member is an ObjectName injected into MessagingPostOfficeService. Better is to expose a property of type org.jgroups.ChannelFactory and inject an instance of that. That's the POJI interface for an object that provides JGroups channels.
> Background:
> JGroups intends to remove the legacy org.jgroups.jmx.JChannelFactoryMBean interface that is being invoked in the above code. That code is reflection based, so as long as a service that exposes the MUX_OPERATION signature is registered in JMX, the call will still work. AS 5 will ensure that such a service is registered, without using the org.jgroups.jmx.JChannelFactoryMBean itself. So, the above JMX-based code will work, and this isn't an urgent issue. But, this usage does tie JBM clustering to JMX, rather than allowing simple MC integration or even manual instantiation.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the jboss-jira
mailing list