[jboss-remoting-commits] JBoss Remoting SVN: r6529 - remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting.

jboss-remoting-commits at lists.jboss.org jboss-remoting-commits at lists.jboss.org
Tue Dec 9 01:18:55 EST 2014


Author: ron.sigal at jboss.com
Date: 2014-12-09 01:18:54 -0500 (Tue, 09 Dec 2014)
New Revision: 6529

Modified:
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ConnectionValidator.java
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Lease.java
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/LeasePinger.java
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ServerInvoker.java
   remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Version.java
Log:
Cleaned up System.out.println() and log.info() calls.

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ConnectionValidator.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ConnectionValidator.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ConnectionValidator.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -360,7 +360,7 @@
          {
             isValid = false;
 
-            log.info(this + ": tieToLease: " + tieToLease + ", client lease period: " + client.getLeasePeriod());
+            log.trace(this + ": tieToLease: " + tieToLease + ", client lease period: " + client.getLeasePeriod());
 //            if (tieToLease && client.getLeasePeriod() > 0)
             if (tieToLease && clientLeasePeriod > 0)
             {
@@ -469,9 +469,9 @@
          if (s.size() == 0)
          {
             listeners.remove(listener);
-            log.info(this + " removed " + listener);
+            log.debug(this + " removed " + listener);
          }
-         log.info(this + " listeners.size(): " + listeners.size());
+         log.trace(this + " listeners.size(): " + listeners.size());
          if (listeners.size() == 0)
          {
             stop();
@@ -677,7 +677,7 @@
                try
                {
                   tieToLease = Boolean.valueOf(((String) o)).booleanValue();
-                  log.info(this + " tieToLease: " + tieToLease);
+                  log.trace(this + " tieToLease: " + tieToLease);
                }
                catch (Exception e)
                {

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Lease.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Lease.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Lease.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -87,7 +87,7 @@
       }
       this.leaseWindow = leasePeriod * 2;
       this.clientLeases = clientLeases;
-      log.info("clientLeases: " + clientLeases);
+      log.trace("clientLeases: " + clientLeases);
    }
 
 

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/LeasePinger.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/LeasePinger.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/LeasePinger.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -321,7 +321,7 @@
 
    public long getLeasePeriod(String sessionID)
    {
-      log.info(this + ": timerTask: " + timerTask + ", clientSessionIds.containsKey(sessionID): " + clientSessionIds.containsKey(sessionID));
+      log.trace(this + ": timerTask: " + timerTask + ", clientSessionIds.containsKey(sessionID): " + clientSessionIds.containsKey(sessionID));
       if (timerTask == null)
       {
          return -1;

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -472,7 +472,7 @@
    {
       synchronized(clientLeaseLock)
       {
-         log.info(this + ": leasePinger: " + leasePinger);
+         log.trace(this + ": leasePinger: " + leasePinger);
          if(leasePinger == null)
          {
             return -1;

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ServerInvoker.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ServerInvoker.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/ServerInvoker.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -449,7 +449,7 @@
 
    public void removeConnectionListener(ConnectionListener listener)
    {
-      System.out.println(this + " removing ConnecttionListener " + listener);
+      log.trace(this + " removing ConnecttionListener " + listener);
       if(connectionNotifier != null)
       {
          connectionNotifier.removeListener(listener);
@@ -467,7 +467,7 @@
                String sessionId = (String)itr.next();
                Lease clientLease = (Lease)clientLeases.get(sessionId);
                clientLease.terminateLease(sessionId);
-               System.out.println(this + " terminating " + clientLease);
+               log.trace(this + " terminating " + clientLease);
             }
             clientLeases.clear();
          }
@@ -2066,13 +2066,16 @@
          }
          if(clientSessionId != null)
          {
-            if(trace) { log.trace("Getting lease for invoker session id: " + clientSessionId); }
-            System.out.println(this + ": existing leases:");
-            Iterator it = clientLeases.keySet().iterator();
-            while (it.hasNext())
-            {
-               Object key = it.next();
-               log.info("  " + clientLeases.get(key));
+            if(trace)
+            { 
+               log.trace(this + ": Getting lease for invoker session id: " + clientSessionId);
+               log.trace(this + ": existing leases:");
+               Iterator it = clientLeases.keySet().iterator();
+               while (it.hasNext())
+               {
+                  Object key = it.next();
+                  log.trace("  " + clientLeases.get(key));
+               }
             }
             
             Lease clientLease = null;
@@ -2114,7 +2117,7 @@
                      if (trace) log.trace("terminating invoker lease: " + clientLease);
                      clientLease.terminateLeaseUponFailure(clientSessionId);
                      clientLeases.remove(clientSessionId);
-log.info(this + " clientLeases: " + clientLeases);
+                     log.trace(this + " clientLeases: " + clientLeases);
                      Lease newClientLease = new Lease(clientSessionId, leasePeriod,
                                                       locator.getLocatorURI(),
                                                       invocation.getRequestPayload(),

Modified: remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Version.java
===================================================================
--- remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Version.java	2014-11-20 06:08:05 UTC (rev 6528)
+++ remoting2/branches/2.5.4.SP3-01190444/src/main/org/jboss/remoting/Version.java	2014-12-09 06:18:54 UTC (rev 6529)
@@ -40,7 +40,7 @@
    public static final byte VERSION_2 = 2;
    public static final byte VERSION_2_2 = 22;
 
-   public static final String VERSION = "2.5.4.SP3-0119044 (Flounder) [01190444: 11/6/14: 01:39";
+   public static final String VERSION = "2.5.4.SP3-0119044 (Flounder) [01190444: 12/09/14: 01:56";
    private static final byte byteVersion = VERSION_2_2;
    private static byte defaultByteVersion = byteVersion;
    private static boolean performVersioning = true;



More information about the jboss-remoting-commits mailing list