You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2008/12/24 16:59:04 UTC

svn commit: r729323 - in /tomcat/trunk/java/org/apache/catalina/tribes/transport/bio: MultipointBioSender.java PooledMultiSender.java util/FastQueue.java util/SingleRemoveSynchronizedAddLock.java

Author: markt
Date: Wed Dec 24 07:59:03 2008
New Revision: 729323

URL: http://svn.apache.org/viewvc?rev=729323&view=rev
Log:
Generics for o.a.c.tribes.transport.bio
Fix various Eclipse warnings (unused code etc)

Modified:
    tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/MultipointBioSender.java
    tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/PooledMultiSender.java
    tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/FastQueue.java
    tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/SingleRemoveSynchronizedAddLock.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/MultipointBioSender.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/MultipointBioSender.java?rev=729323&r1=729322&r2=729323&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/MultipointBioSender.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/MultipointBioSender.java Wed Dec 24 07:59:03 2008
@@ -41,7 +41,8 @@
     }
     
     protected long selectTimeout = 1000; 
-    protected HashMap bioSenders = new HashMap();
+    protected HashMap<Member, BioSender> bioSenders =
+        new HashMap<Member, BioSender>();
 
     public synchronized void sendMessage(Member[] destination, ChannelMessage msg) throws ChannelException {
         byte[] data = XByteBuffer.createDataPackage((ChannelData)msg);
@@ -65,10 +66,10 @@
         BioSender[] result = new BioSender[destination.length];
         for ( int i=0; i<destination.length; i++ ) {
             try {
-                BioSender sender = (BioSender) bioSenders.get(destination[i]);
+                BioSender sender = bioSenders.get(destination[i]);
                 if (sender == null) {
                     sender = new BioSender();
-                    sender.transferProperties(this,sender);
+                    AbstractSender.transferProperties(this,sender);
                     sender.setDestination(destination[i]);
                     bioSenders.put(destination[i], sender);
                 }
@@ -96,7 +97,7 @@
         for (int i=0; i<members.length; i++ ) {
             Member mbr = (Member)members[i];
             try {
-                BioSender sender = (BioSender)bioSenders.get(mbr);
+                BioSender sender = bioSenders.get(mbr);
                 sender.disconnect();
             }catch ( Exception e ) {
                 if ( x == null ) x = new ChannelException(e);
@@ -113,7 +114,7 @@
 
     public void remove(Member member) {
         //disconnect senders
-        BioSender sender = (BioSender)bioSenders.remove(member);
+        BioSender sender = bioSenders.remove(member);
         if ( sender != null ) sender.disconnect();
     }
 
@@ -131,9 +132,9 @@
     public boolean keepalive() {
         //throw new UnsupportedOperationException("Method ParallelBioSender.checkKeepAlive() not implemented");
         boolean result = false;
-        Map.Entry[] entries = (Map.Entry[])bioSenders.entrySet().toArray(new Map.Entry[bioSenders.size()]);
+        Map.Entry<Member,BioSender>[] entries = bioSenders.entrySet().toArray(new Map.Entry[bioSenders.size()]);
         for ( int i=0; i<entries.length; i++ ) {
-            BioSender sender = (BioSender)entries[i].getValue();
+            BioSender sender = entries[i].getValue();
             if ( sender.keepalive() ) {
                 bioSenders.remove(entries[i].getKey());
             }

Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/PooledMultiSender.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/PooledMultiSender.java?rev=729323&r1=729322&r2=729323&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/PooledMultiSender.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/PooledMultiSender.java Wed Dec 24 07:59:03 2008
@@ -16,6 +16,7 @@
  */
 package org.apache.catalina.tribes.transport.bio;
 
+import org.apache.catalina.tribes.transport.AbstractSender;
 import org.apache.catalina.tribes.transport.DataSender;
 import org.apache.catalina.tribes.transport.PooledSender;
 import org.apache.catalina.tribes.Member;
@@ -65,7 +66,7 @@
      */
     public DataSender getNewDataSender() {
         MultipointBioSender sender = new MultipointBioSender();
-        sender.transferProperties(this,sender);
+        AbstractSender.transferProperties(this,sender);
         return sender;
     }
 

Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/FastQueue.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/FastQueue.java?rev=729323&r1=729322&r2=729323&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/FastQueue.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/FastQueue.java Wed Dec 24 07:59:03 2008
@@ -61,11 +61,7 @@
      */
     private boolean checkLock = false;
 
-    /**
-     * protocol the thread wait times
-     */
-    private boolean timeWait = false;
-
+    
     private boolean inAdd = false;
 
     private boolean inRemove = false;
@@ -99,11 +95,6 @@
     private int maxSize = 0;
 
     /**
-     *  avg size sample interval
-     */
-    private int sampleInterval = 100;
-
-    /**
      * Generate Queue SingleRemoveSynchronizedAddLock and set add and wait
      * Timeouts
      */
@@ -251,7 +242,6 @@
      */
     public boolean add(ChannelMessage msg, Member[] destination, InterceptorPayload payload) {
         boolean ok = true;
-        long time = 0;
 
         if (!enabled) {
             if (log.isInfoEnabled())
@@ -259,9 +249,6 @@
             return false;
         }
 
-        if (timeWait) {
-            time = System.currentTimeMillis();
-        }
         lock.lockAdd();
         try {
             if (log.isTraceEnabled()) {
@@ -327,7 +314,6 @@
     public LinkObject remove() {
         LinkObject element;
         boolean gotLock;
-        long time = 0;
 
         if (!enabled) {
             if (log.isInfoEnabled())
@@ -335,9 +321,6 @@
             return null;
         }
 
-        if (timeWait) {
-            time = System.currentTimeMillis();
-        }
         gotLock = lock.lockRemove();
         try {
 
@@ -381,9 +364,6 @@
                 log.trace("FastQueue.remove: remove ending with size " + size);
             }
 
-            if (timeWait) {
-                time = System.currentTimeMillis();
-            }
         } finally {
             lock.unlockRemove();
         }

Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/SingleRemoveSynchronizedAddLock.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/SingleRemoveSynchronizedAddLock.java?rev=729323&r1=729322&r2=729323&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/SingleRemoveSynchronizedAddLock.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/util/SingleRemoveSynchronizedAddLock.java Wed Dec 24 07:59:03 2008
@@ -183,7 +183,7 @@
                 try {
                     wait(addWaitTimeout);
                 } catch ( InterruptedException e ) {
-                    Thread.currentThread().interrupted();
+                    Thread.interrupted();
                 }
             } while ( addLocked || removeLocked );
         }
@@ -205,7 +205,7 @@
                 try {
                     wait(removeWaitTimeout);
                 } catch ( InterruptedException e ) {
-                    Thread.currentThread().interrupted();
+                    Thread.interrupted();
                 }
             } while ( ( addLocked || ! dataAvailable ) && removeEnabled );
             remover=null;



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