Author: kfujino
Date: Wed Aug  4 08:59:48 2010
New Revision: 982165

URL: http://svn.apache.org/viewvc?rev=982165&view=rev
Log:
Remove domainReplication attribute from ClusterManager. 

Modified:
    tomcat/trunk/java/org/apache/catalina/ha/CatalinaCluster.java
    tomcat/trunk/java/org/apache/catalina/ha/ClusterManager.java
    
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java
    tomcat/trunk/java/org/apache/catalina/ha/session/BackupManager.java
    tomcat/trunk/java/org/apache/catalina/ha/session/DeltaManager.java
    tomcat/trunk/java/org/apache/catalina/ha/session/DeltaSession.java
    tomcat/trunk/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java
    tomcat/trunk/java/org/apache/catalina/ha/session/mbeans-descriptors.xml
    tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java
    tomcat/trunk/java/org/apache/catalina/ha/tcp/SimpleTcpCluster.java
    tomcat/trunk/java/org/apache/catalina/ha/tcp/mbeans-descriptors.xml
    tomcat/trunk/webapps/docs/changelog.xml
    tomcat/trunk/webapps/docs/config/cluster-interceptor.xml
    tomcat/trunk/webapps/docs/config/cluster-manager.xml

Modified: tomcat/trunk/java/org/apache/catalina/ha/CatalinaCluster.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/CatalinaCluster.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/CatalinaCluster.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ha/CatalinaCluster.java Wed Aug  4 
08:59:48 2010
@@ -77,13 +77,6 @@ public interface CatalinaCluster extends
      * @param dest Member
      */
     public void send(ClusterMessage msg, Member dest);
-    
-    /**
-     * Sends a message to a all members at local cluster domain
-     *
-     * @param msg ClusterMessage
-     */
-    public void sendClusterDomain(ClusterMessage msg);
 
     /**
      * Returns that cluster has members.

Modified: tomcat/trunk/java/org/apache/catalina/ha/ClusterManager.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/ClusterManager.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/ClusterManager.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ha/ClusterManager.java Wed Aug  4 
08:59:48 2010
@@ -78,18 +78,6 @@ public interface ClusterManager extends 
    public CatalinaCluster getCluster();
 
    public void setCluster(CatalinaCluster cluster);
-   
-   /**
-    * @return Manager send only to same cluster domain.
-    * @since 5.5.10
-    */
-   public boolean doDomainReplication();
-
-   /**
-    * @param domainReplication Flag value.
-    * @since 5.5.10
-    */
-   public void setDomainReplication(boolean domainReplication);
 
    public ReplicationStream getReplicationStream(byte[] data) throws 
IOException;
 

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java 
Wed Aug  4 08:59:48 2010
@@ -201,7 +201,7 @@ public class ClusterSingleSignOn
 
             msg.setAction(SingleSignOnMessage.ADD_SESSION);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
 
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
@@ -241,7 +241,7 @@ public class ClusterSingleSignOn
 
             msg.setAction(SingleSignOnMessage.DEREGISTER_SESSION);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
                                    + msg.getAction());
@@ -275,7 +275,7 @@ public class ClusterSingleSignOn
                                         ssoId, null);
             msg.setAction(SingleSignOnMessage.LOGOUT_SESSION);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
                                    + msg.getAction());
@@ -317,7 +317,7 @@ public class ClusterSingleSignOn
             msg.setUsername(username);
             msg.setPassword(password);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
                                    + msg.getAction());
@@ -375,7 +375,7 @@ public class ClusterSingleSignOn
             msg.setUsername(username);
             msg.setPassword(password);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
                                    + msg.getAction());
@@ -415,7 +415,7 @@ public class ClusterSingleSignOn
 
             msg.setAction(SingleSignOnMessage.REMOVE_SESSION);
 
-            cluster.sendClusterDomain(msg);
+            cluster.send(msg);
             if (containerLog.isDebugEnabled())
                 containerLog.debug("SingleSignOnMessage Send with action "
                                    + msg.getAction());

Modified: tomcat/trunk/java/org/apache/catalina/ha/session/BackupManager.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/BackupManager.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/BackupManager.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/BackupManager.java Wed Aug 
 4 08:59:48 2010
@@ -82,16 +82,6 @@ public class BackupManager extends Clust
     public void messageDataReceived(ClusterMessage msg) {
     }
 
-    public boolean doDomainReplication() {
-        return false;
-    }
-
-    /**
-     * @param sendClusterDomainOnly The sendClusterDomainOnly to set.
-     */
-    public void setDomainReplication(boolean sendClusterDomainOnly) {
-    }
-
     public void setExpireSessionsOnShutdown(boolean expireSessionsOnShutdown)
     {
         mExpireSessionsOnShutdown = expireSessionsOnShutdown;

Modified: tomcat/trunk/java/org/apache/catalina/ha/session/DeltaManager.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/DeltaManager.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/DeltaManager.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/DeltaManager.java Wed Aug  
4 08:59:48 2010
@@ -100,7 +100,6 @@ public class DeltaManager extends Cluste
     private volatile boolean stateTransfered = false ;
     private int stateTransferTimeout = 60;
     private boolean sendAllSessions = true;
-    private boolean sendClusterDomainOnly = true ;
     private int sendAllSessionsSize = 1000 ;
     
     /**
@@ -346,20 +345,6 @@ public class DeltaManager extends Cluste
     }
     
     /**
-     * @return Returns the sendClusterDomainOnly.
-     */
-    public boolean doDomainReplication() {
-        return sendClusterDomainOnly;
-    }
-    
-    /**
-     * @param sendClusterDomainOnly The sendClusterDomainOnly to set.
-     */
-    public void setDomainReplication(boolean sendClusterDomainOnly) {
-        this.sendClusterDomainOnly = sendClusterDomainOnly;
-    }
-
-    /**
      * @return Returns the stateTimestampDrop.
      */
     public boolean isStateTimestampDrop() {
@@ -511,10 +496,7 @@ public class DeltaManager extends Cluste
      */
     protected void send(SessionMessage msg) {
         if(cluster != null) {
-            if(doDomainReplication())
-                cluster.sendClusterDomain(msg);
-            else
-                cluster.send(msg);
+            cluster.send(msg);
         }
     }
 
@@ -1234,7 +1216,7 @@ public class DeltaManager extends Cluste
      *            requesting node
      */
     protected void messageReceived(SessionMessage msg, Member sender) {
-        if(doDomainReplication() && !checkSenderDomain(msg,sender)) {
+        if(!checkSenderDomain(msg,sender)) {
             return;
         }
         ClassLoader contextLoader = 
Thread.currentThread().getContextClassLoader();
@@ -1492,7 +1474,6 @@ public class DeltaManager extends Cluste
         result.notifySessionListenersOnReplication = 
notifySessionListenersOnReplication;
         result.stateTransferTimeout = stateTransferTimeout;
         result.sendAllSessions = sendAllSessions;
-        result.sendClusterDomainOnly = sendClusterDomainOnly ;
         result.sendAllSessionsSize = sendAllSessionsSize;
         result.sendAllSessionsWaitTime = sendAllSessionsWaitTime ; 
         result.receiverQueue = receiverQueue ;

Modified: tomcat/trunk/java/org/apache/catalina/ha/session/DeltaSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/DeltaSession.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/DeltaSession.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/DeltaSession.java Wed Aug  
4 08:59:48 2010
@@ -402,11 +402,7 @@ public class DeltaSession extends Standa
             CatalinaCluster cluster = dmanager.getCluster();
             ClusterMessage msg = dmanager.requestCompleted(expiredId, true);
             if (msg != null) {
-                if(dmanager.doDomainReplication()) {
-                    cluster.sendClusterDomain(msg);
-                } else {
-                    cluster.send(msg);
-                }
+                cluster.send(msg);
             }
         }
 

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/JvmRouteBinderValve.java 
Wed Aug  4 08:59:48 2010
@@ -421,10 +421,7 @@ public class JvmRouteBinderValve extends
         Context context = request.getContext();
         msg.setContextPath(context.getPath());
         msg.setHost(context.getParent().getName());
-        if(manager.doDomainReplication())
-            cluster.sendClusterDomain(msg);
-        else
-            cluster.send(msg);
+        cluster.send(msg);
     }
 
     /**

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/session/mbeans-descriptors.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/mbeans-descriptors.xml?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/mbeans-descriptors.xml 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/mbeans-descriptors.xml Wed 
Aug  4 08:59:48 2010
@@ -275,11 +275,6 @@
       description="Total number of sessions created by this manager"
       type="long"/>
     <attribute
-      name="sendClusterDomainOnly"
-      is="true"
-      description="The sendClusterDomainOnly flag send sessions only to 
members as same cluster domain"
-      type="boolean"/>
-    <attribute
       name="sessionIdLength"
       description="The session id length (in bytes) of Sessions created by 
this Manager"
       type="int"/>

Modified: tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java Wed Aug  
4 08:59:48 2010
@@ -539,11 +539,7 @@ public class ReplicationValve
     protected void send(ClusterManager manager, CatalinaCluster cluster, 
String sessionId) {
         ClusterMessage msg = manager.requestCompleted(sessionId);
         if (msg != null) {
-            if(manager.doDomainReplication()) {
-                cluster.sendClusterDomain(msg);
-            } else {
-                cluster.send(msg);
-            }
+            cluster.send(msg);
             if(doStatistics())
                 nrOfSendRequests++;
         }

Modified: tomcat/trunk/java/org/apache/catalina/ha/tcp/SimpleTcpCluster.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/tcp/SimpleTcpCluster.java?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/tcp/SimpleTcpCluster.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/tcp/SimpleTcpCluster.java Wed Aug  
4 08:59:48 2010
@@ -769,16 +769,6 @@ public class SimpleTcpCluster extends Li
     public void send(ClusterMessage msg) {
         send(msg, null);
     }
-
-    /**
-     * send message to all cluster members same cluster domain
-     * 
-     * @see 
org.apache.catalina.ha.CatalinaCluster#send(org.apache.catalina.ha.ClusterMessage)
-     */
-    public void sendClusterDomain(ClusterMessage msg) {
-        send(msg,null);
-    } 
-
     
     /**
      * send a cluster message to one member

Modified: tomcat/trunk/java/org/apache/catalina/ha/tcp/mbeans-descriptors.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/tcp/mbeans-descriptors.xml?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/tcp/mbeans-descriptors.xml 
(original)
+++ tomcat/trunk/java/org/apache/catalina/ha/tcp/mbeans-descriptors.xml Wed Aug 
 4 08:59:48 2010
@@ -79,16 +79,6 @@
         type="org.apache.catalina.ha.ClusterMessage"/>
     </operation>
     <operation
-      name="sendClusterDomain"
-      description="send message to all cluster members with same domain"
-      impact="ACTION"
-      returnType="void">
-      <parameter
-        name="message"
-        description="replication message"
-        type="org.apache.catalina.ha.ClusterMessage"/>
-    </operation>
-    <operation
       name="start"
       description="Start the cluster"
       impact="ACTION"

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Wed Aug  4 08:59:48 2010
@@ -48,6 +48,15 @@
       </update>
     </changelog>
   </subsection>
+  <subsection name="Cluster">
+    <changelog>
+      <fix>
+        Remove domainReplication attribute from ClusterManager. 
+        If you send session to only same domain, use DomainFilterInterceptor. 
+        (kfujino)
+      </fix>
+    </changelog>
+  </subsection>
   <subsection name="Web applications">
     <changelog>
       <fix>

Modified: tomcat/trunk/webapps/docs/config/cluster-interceptor.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/cluster-interceptor.xml?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/config/cluster-interceptor.xml (original)
+++ tomcat/trunk/webapps/docs/config/cluster-interceptor.xml Wed Aug  4 
08:59:48 2010
@@ -96,6 +96,17 @@
    </attributes>
   </subsection>
 
+  <subsection 
name="org.apache.catalina.tribes.group.interceptors.DomainFilterInterceptor 
Attributes">
+   <attributes>
+     <attribute name="domain" required="true">
+       The logical cluster domain that this Interceptor accepts.
+       Two different type of values are possible:<br/>
+       1. Regular string values like &quot;staging-domain&quot; or 
&quot;tomcat-cluster&quot; will be converted into bytes 
+       using ISO-8859-1 encoding.<br/>
+       2. byte array in string form, for example {216,123,12,3}<br/>
+     </attribute>
+   </attributes>
+  </subsection>
   <subsection 
name="org.apache.catalina.tribes.group.interceptors.MessageDispatch15Interceptor
 Attributes">
    <attributes>
      <attribute name="className" required="true">

Modified: tomcat/trunk/webapps/docs/config/cluster-manager.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/cluster-manager.xml?rev=982165&r1=982164&r2=982165&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/config/cluster-manager.xml (original)
+++ tomcat/trunk/webapps/docs/config/cluster-manager.xml Wed Aug  4 08:59:48 
2010
@@ -87,13 +87,6 @@
   </subsection> 
   <subsection name="org.apache.catalina.ha.session.DeltaManager Attributes">
     <attributes>
-      <attribute name="domainReplication" required="false">
-        Set to true if you wish sessions to be replicated only to members that
-        have the same logical domain set. If set to false, session replication
-        will ignore the domain setting the 
-        <code><a href="cluster-membership.html">&lt;Membership&gt;</a></code>
-        element.
-      </attribute>
       <attribute name="expireSessionsOnShutdown" required="false">
         When a web application is being shutdown, Tomcat issues an expire call
         to each session to notify all the listeners. If you wish for all



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to