[jboss-cvs] JBossAS SVN: r58566 - trunk/cluster/src/main/org/jboss/ha/framework/server

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2006-11-18 06:27:50 -0500 (Sat, 18 Nov 2006)
New Revision: 58566

Modified:
   trunk/cluster/src/main/org/jboss/ha/framework/server/ClusterPartitionMBean.java
Log:
Rationalize interface; use JBC 2.0; don't expose setters for immutable stuff we configure via -beans.xml

Modified: trunk/cluster/src/main/org/jboss/ha/framework/server/ClusterPartitionMBean.java
===================================================================
--- trunk/cluster/src/main/org/jboss/ha/framework/server/ClusterPartitionMBean.java	2006-11-18 11:26:45 UTC (rev 58565)
+++ trunk/cluster/src/main/org/jboss/ha/framework/server/ClusterPartitionMBean.java	2006-11-18 11:27:50 UTC (rev 58566)
@@ -26,7 +26,7 @@
 
 import javax.management.ObjectName;
 
-import org.jboss.cache.TreeCacheMBean;
+import org.jboss.cache.Cache;
 import org.jboss.ha.framework.interfaces.DistributedState;
 import org.jboss.ha.framework.interfaces.HAPartition;
 import org.jboss.mx.util.ObjectNameFactory;
@@ -53,21 +53,18 @@
     * a partition with the same name are clustered together.
     */
    String getPartitionName();
-   void setPartitionName(String newName);
 
    /**
     * Uniquely identifies this node. MUST be unique accros the whole cluster!
     * Cannot be changed once the partition has been started (otherwise an exception is thrown)
     */
-   String getNodeName();
-   void setNodeName(String node) throws Exception;
-
+   String getNodeUniqueId();
+   
    /**
     * The node address used to generate the node name
     */
    InetAddress getNodeAddress();
-   void setNodeAddress(InetAddress address);
-
+   
    /** The version of JGroups this is running on */
    String getJGroupsVersion();
 
@@ -92,7 +89,6 @@
     * Determine if deadlock detection is enabled
     */
    boolean getDeadlockDetection();
-   void setDeadlockDetection(boolean doit);
    
    /**
     * Returns whether this partition will synchronously notify any 
@@ -132,34 +128,16 @@
    /**
     * Access to the DistributedState implementation
     */
-   DistributedState getDistributedState();
+   DistributedState getDistributedStateService();
    
    /**
-    * Sets the DistributedState used by this partition.
-    */
-   void setDistributedState(DistributedState state);
-   
-   /**
     * Gets the TreeCache used by this partition for state management.
     */
-   TreeCacheMBean getTreeCache();
+   Cache getClusteredCache();
    
    /**
-    * Sets the TreeCache used by this partition for state management.
-    * 
-    * <strong>NOTE:</strong> The cache must be configured to use a JGroups
-    * multiplexer channel.
-    * @param cache the cache
-    * 
-    * @throws IllegalArgumentException if {@TreeCacheMBean#getMultiplexer
-    * @throws NullPointerException if cache is <code>null</code>
+    * Gets the JGroups multiplexer channel factory used by ClusterPartition.
     */
-   void setTreeCache(TreeCacheMBean cache);
-   
-   /**
-    * Injects JGroups multiplexer channel into ClusterPartition.
-    * This attribute is optional; if not provided, a default JGroups JChannel will be used
-    */
    JChannelFactoryMBean getMultiplexer();
    
    /**