[jboss-cvs] JBossCache/src/org/jboss/cache/loader/tcp ...

Elias Ross genman at noderunner.net
Thu Jan 25 16:54:05 EST 2007


  User: genman  
  Date: 07/01/25 16:54:05

  Modified:    src/org/jboss/cache/loader/tcp  TcpCacheServer.java
  Log:
  JBCACHE-952 -  Replace calls to Throwable.printStackTace with logging
  
  Revision  Changes    Path
  1.27      +11 -10    JBossCache/src/org/jboss/cache/loader/tcp/TcpCacheServer.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: TcpCacheServer.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/loader/tcp/TcpCacheServer.java,v
  retrieving revision 1.26
  retrieving revision 1.27
  diff -u -b -r1.26 -r1.27
  --- TcpCacheServer.java	18 Jan 2007 16:55:19 -0000	1.26
  +++ TcpCacheServer.java	25 Jan 2007 21:54:05 -0000	1.27
  @@ -36,7 +36,7 @@
    * TCP-IP based CacheServer, setCache TcpDelegatingCacheLoader with host and port of this server
    *
    * @author Bela Ban
  - * @version $Id: TcpCacheServer.java,v 1.26 2007/01/18 16:55:19 msurtani Exp $
  + * @version $Id: TcpCacheServer.java,v 1.27 2007/01/25 21:54:05 genman Exp $
    */
   public class TcpCacheServer implements TcpCacheServerMBean
   {
  @@ -55,7 +55,7 @@
       * whether or not to start the server thread as a daemon.  Should be false if started from the command line, true if started as an MBean.
       */
      boolean daemon = true;
  -   static Log mylog = LogFactory.getLog(TcpCacheServer.class);
  +   static Log log = LogFactory.getLog(TcpCacheServer.class);
   
   
      public TcpCacheServer()
  @@ -156,7 +156,7 @@
   
         srv_sock = new ServerSocket(port, 10, bind_addr);
         System.out.println("TcpCacheServer listening on : " + srv_sock.getInetAddress() + ":" + srv_sock.getLocalPort());
  -      mylog.info("TcpCacheServer listening on : " + srv_sock.getInetAddress() + ":" + srv_sock.getLocalPort());
  +      log.info("TcpCacheServer listening on : " + srv_sock.getInetAddress() + ":" + srv_sock.getLocalPort());
   
         running = true;
   
  @@ -180,16 +180,16 @@
                  {
                     // this is because of the stop() lifecycle method being called.
                     // ignore.
  -                  mylog.info("Shutting down TcpCacheServer");
  +                  log.info("Shutting down TcpCacheServer");
                  }
                  else
                  {
  -                  mylog.error("Caught exception! Shutting down server thread.", se);
  +                  log.error("Caught exception! Shutting down server thread.", se);
                  }
               }
               catch (IOException e)
               {
  -               mylog.error("Caught exception! Shutting down server thread.", e);
  +               log.error("Caught exception! Shutting down server thread.", e);
               }
            }
         };
  @@ -319,7 +319,7 @@
               }
               catch (IOException e)
               {
  -               mylog.debug("Client closed socket");
  +               log.debug("Client closed socket");
                  close();
                  break;
               }
  @@ -428,13 +428,14 @@
                        output.writeObject(Boolean.TRUE);
                        break;
                     default:
  -                     mylog.error("Operation " + op + " unknown");
  +                     log.error("Operation " + op + " unknown");
                        break;
                  }
                  output.flush();
               }
               catch (Exception e)
               {
  +               log.debug(e, e);
                  try
                  {
                     output.writeObject(e);
  @@ -442,7 +443,7 @@
                  }
                  catch (IOException e1)
                  {
  -                  e1.printStackTrace();
  +                  log.error(e1, e1);
                  }
               }
            }
  @@ -488,7 +489,7 @@
                     c.move(m.getFqn(), m.getFqn2());
                     break;
                  default:
  -                  mylog.error("modification type " + m.getType() + " not known");
  +                  log.error("modification type " + m.getType() + " not known");
                     break;
               }
            }
  
  
  



More information about the jboss-cvs-commits mailing list