[mod_cluster-issues] [JBoss JIRA] Created: (MODCLUSTER-110) Split ModClusterServiceMBean.ping(String) into 3 methods

Paul Ferraro (JIRA) jira-events at lists.jboss.org
Tue Oct 27 13:07:05 EDT 2009


Split ModClusterServiceMBean.ping(String) into 3 methods
--------------------------------------------------------

                 Key: MODCLUSTER-110
                 URL: https://jira.jboss.org/jira/browse/MODCLUSTER-110
             Project: mod_cluster
          Issue Type: Feature Request
            Reporter: Paul Ferraro
            Assignee: Jean-Frederic Clere
             Fix For: 1.1.0.CR1


Currently the ModClusterServiceMBean.ping(String) method does 3 things:
1. If parameter is null, send a PING command that determines the accessibility/health of each proxy.
2. If parameter is not a url, interpret the parameter as a jvm route and send a PING command that determines the accessibility of the node, configured with the specified jvm route, from each proxy.
3. If parameter is as a url, interpret the parameter as a url and send a PING command that determines the accessibility of the node (which may or may not be configured in the proxy already) containing a connector matching the protocol, host, and port of the url, from each proxy .

Rather than 1 multi-purpose function - this method should be split in to 3, corresponding to the 3 functions identified above:
Map<InetSocketAddress, String> ping();
Map<InetSocketAddress, String> ping(String jvmRoute);
Map<InetSocketAddress, String> ping(String protocol, String host, int port);

or perhaps:
Map<InetSocketAddress, String> pingProxies();
Map<InetSocketAddress, String> pingNode(String jvmRoute);
Map<InetSocketAddress, String> pingConector(String url);


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the mod_cluster-issues mailing list