[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting/transport/socket ...

[Ron Sigal]

  User: rsigal  
  Date: 07/08/06 00:01:37

  Modified:    src/main/org/jboss/remoting/transport/socket  Tag:
                        remoting_2_2_0_GA SocketServerInvoker.java
  Log:
  JBREM-766: Added shutdown() to ServerSocketRefresh thread so stopping thread no longer relies on interrupt.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.30.2.12.2.1 +15 -6     JBossRemoting/src/main/org/jboss/remoting/transport/socket/SocketServerInvoker.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: SocketServerInvoker.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/transport/socket/SocketServerInvoker.java,v
  retrieving revision 1.30.2.12
  retrieving revision 1.30.2.12.2.1
  diff -u -b -r1.30.2.12 -r1.30.2.12.2.1
  --- SocketServerInvoker.java	11 Mar 2007 20:49:11 -0000	1.30.2.12
  +++ SocketServerInvoker.java	6 Aug 2007 04:01:37 -0000	1.30.2.12.2.1
  @@ -49,7 +49,7 @@
    * @author <a href="mailto:tom.elrod at jboss.com">Tom Elrod</a>
    * @author <a href="mailto:ovidiu at jboss.org">Ovidiu Feodorov</a>
    *
  - * @version $Revision: 1.30.2.12 $
  + * @version $Revision: 1.30.2.12.2.1 $
    * @jmx:mbean
    */
   public class SocketServerInvoker extends ServerInvoker implements Runnable, SocketServerInvokerMBean
  @@ -554,7 +554,7 @@
         }
         finally
         {
  -         thread.interrupt();
  +         thread.shutdown();
         }
      }
   
  @@ -671,6 +671,8 @@
       */
      public class ServerSocketRefresh extends Thread
      {  
  +      private boolean running = true;
  +      
         public ServerSocketRefresh()
         {
            super("ServerSocketRefresh");
  @@ -678,10 +680,8 @@
         
         public void run()
         {
  -         while(true)
  +         while(running)
            {
  -            if(isInterrupted())break;
  -            
               synchronized (serverSocketFactoryLock)
               {  
                  if(newServerSocketFactory)
  @@ -706,7 +706,6 @@
                  }
                  catch (InterruptedException e)
                  {
  -                  break;
                  }
               }
            }
  @@ -726,6 +725,16 @@
               log.trace("passed through ServerSocketRefresh.release()");
            }
         }
  +      
  +      public void shutdown()
  +      {
  +         running = false;
  +         
  +         synchronized (serverSocketFactoryLock)
  +         {
  +            serverSocketFactoryLock.notify();
  +         }
  +      }
      }
   
      /**