[jboss-cvs] JBossAS SVN: r67151 - trunk/cluster/src/main/org/jboss/ha/framework/interfaces.

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

Author: bstansberry at jboss.com
Date: 2007-11-15 17:21:07 -0500 (Thu, 15 Nov 2007)
New Revision: 67151

Modified:
   trunk/cluster/src/main/org/jboss/ha/framework/interfaces/LoadBalancePolicy.java
Log:
[JBAS-4977] Extract a parent interface that doesn't leak legacy classes

Modified: trunk/cluster/src/main/org/jboss/ha/framework/interfaces/LoadBalancePolicy.java
===================================================================
--- trunk/cluster/src/main/org/jboss/ha/framework/interfaces/LoadBalancePolicy.java	2007-11-15 22:18:42 UTC (rev 67150)
+++ trunk/cluster/src/main/org/jboss/ha/framework/interfaces/LoadBalancePolicy.java	2007-11-15 22:21:07 UTC (rev 67151)
@@ -24,16 +24,15 @@
 import org.jboss.invocation.Invocation;
 
 /**
- * Base interface for load-balancing policies. It is possible to implement many
- * different load-balancing policies by implementing this simple interface and
- * using it in the different clustered services (home interface of SLSB for
- * example)
+ * Extends the {@link org.jboss.ha.client.loadbalance.LoadBalancePolicy parent interface}
+ * by adding support for passing in a legacy (i.e. non-AOP) {@link Invocation}
+ * as an aid in making the choice of targets. 
  *
  * @author <a href="mailto:bill at burkecentral.com">Bill Burke</a>.
  * @author <a href="mailto:sacha.labourey at cogito-info.ch">Sacha Labourey</a>.
  * @version $Revision$
  */
-public interface LoadBalancePolicy extends java.io.Serializable
+public interface LoadBalancePolicy extends org.jboss.ha.client.loadbalance.LoadBalancePolicy
 {
    /** The serialVersionUID
     * @since 1.3.4.2
@@ -50,14 +49,6 @@
     * Called when the stub wishes to know on which node the next invocation must
     * be performed.
     * @param clusterFamily A list of potential target nodes
-    * @return The selected target for the next invocation
-    */   
-   public Object chooseTarget (FamilyClusterInfo clusterFamily);
-
-   /**
-    * Called when the stub wishes to know on which node the next invocation must
-    * be performed.
-    * @param clusterFamily A list of potential target nodes
     * @param routingDecision The actual invocation object if the policy wants
     * to have some kind of invocation-based routing strategy
     * @return The selected target for the next invocation