[jboss-cvs] JBossAS SVN: r62100 - trunk/tomcat/src/main/org/jboss/web/tomcat/service/session.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2007-04-04 16:28:28 -0400 (Wed, 04 Apr 2007)
New Revision: 62100

Modified:
   trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java
   trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java
Log:
Reduce logging levels

Modified: trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java
===================================================================
--- trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java	2007-04-04 20:21:01 UTC (rev 62099)
+++ trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java	2007-04-04 20:28:28 UTC (rev 62100)
@@ -365,8 +365,11 @@
    {
       Session session = findSession(sessionId);
       if(session == null) {
-         log_.debug("getLastAccessedTime(): Session " + sessionId + 
-                    " not found");
+         if (log_.isTraceEnabled())
+         {
+            log_.trace("getLastAccessedTime(): Session " + sessionId + 
+                       " not found");
+         }
          return "";
       }
      return new Date(session.getLastAccessedTime()).toString();
@@ -488,9 +491,9 @@
          // JBCLUSTER-15
          // if session passivation is enabled, passivate sessions instead of expiring them which means
          // they'll be available to the manager for activation after a restart. 
-         if (log_.isDebugEnabled())
+         if (log_.isTraceEnabled())
          {
-             log_.debug("clearSessions(): clear session by expiring or passivating: " + ses);
+             log_.trace("clearSessions(): clear session by expiring or passivating: " + ses);
          }
          boolean notify = true;
          boolean localCall = true;
@@ -600,9 +603,9 @@
           // We are using mod_jk for load balancing. Append the JvmRoute.
           if (useJK_)
           {
-              if (log_.isDebugEnabled())
+              if (log_.isTraceEnabled())
               {
-                  log_.debug("createSession(): useJK is true. Will append JvmRoute: " + this.getJvmRoute());
+                  log_.trace("createSession(): useJK is true. Will append JvmRoute: " + this.getJvmRoute());
               }
               sessionId += "." + this.getJvmRoute();
           }
@@ -610,9 +613,9 @@
 
       session.setId(sessionId); // Setting the id leads to a call to add()
 
-      if (log_.isDebugEnabled())
+      if (log_.isTraceEnabled())
       {
-         log_.debug("Created a ClusteredSession with id: " + sessionId);
+         log_.trace("Created a ClusteredSession with id: " + sessionId);
       }
 
       createdCounter_++;
@@ -719,9 +722,9 @@
          if (activeCounter_ > maxActiveCounter_)
             maxActiveCounter_++;
          
-         if (log_.isDebugEnabled())
+         if (log_.isTraceEnabled())
          {
-            log_.debug("Session with id=" + session.getIdInternal() + " added. " +
+            log_.trace("Session with id=" + session.getIdInternal() + " added. " +
                        "Current active sessions " + activeCounter_);
          }
       }
@@ -736,7 +739,7 @@
 
    private ClusteredSession createEmptyClusteredSession()
    {
-      log_.debug("Creating an empty ClusteredSession");      
+      log_.trace("Creating an empty ClusteredSession");      
 
       ClusteredSession session = null;
       switch (replicationGranularity_)
@@ -838,8 +841,8 @@
          // Make a thread-safe copy of the new id list to work with
          Set ids = new HashSet(unloadedSessions_.keySet());
 
-         if(log_.isDebugEnabled()) {
-            log_.debug("findSessions: loading sessions from distributed cache: " + ids);
+         if(log_.isTraceEnabled()) {
+            log_.trace("findSessions: loading sessions from distributed cache: " + ids);
          }
 
          for(Iterator it = ids.iterator(); it.hasNext();) {
@@ -896,9 +899,9 @@
          if (realId == null)
             return;
 
-         if (log_.isDebugEnabled())
+         if (log_.isTraceEnabled())
          {
-            log_.debug("Removing session from store with id: " + realId);
+            log_.trace("Removing session from store with id: " + realId);
          }
 
          try {
@@ -934,9 +937,9 @@
          String realId = clusterSess.getRealId();
          if (realId == null) return;
 
-         if (log_.isDebugEnabled())
+         if (log_.isTraceEnabled())
          {
-            log_.debug("Removing session from local store with id: " + realId);
+            log_.trace("Removing session from local store with id: " + realId);
          }
 
          try {
@@ -1083,14 +1086,14 @@
             long elapsed = System.currentTimeMillis() - begin;
             stats_.updateLoadStats(realId, elapsed);
 
-            if (log_.isDebugEnabled())
+            if (log_.isTraceEnabled())
             {
-               log_.debug("loadSession(): id= " + realId + ", session=" + session);
+               log_.trace("loadSession(): id= " + realId + ", session=" + session);
             }
          }
-         else if (log_.isDebugEnabled())
+         else if (log_.isTraceEnabled())
          {
-            log_.debug("loadSession(): session " + realId +
+            log_.trace("loadSession(): session " + realId +
                        " not found in distributed cache");
          }
       }
@@ -1287,7 +1290,7 @@
                      {
                         if(log_.isTraceEnabled())
                         {
-                           log_.debug("JBossCacheManager.processExpires() passivating session " + session.getRealId());
+                           log_.trace("JBossCacheManager.processExpires() passivating session " + session.getRealId());
                         }
                         processSessionPassivation(session.getRealId(), this.getContainer().getParent().getName());
                      }

Modified: trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java
===================================================================
--- trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java	2007-04-04 20:21:01 UTC (rev 62099)
+++ trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java	2007-04-04 20:28:28 UTC (rev 62100)
@@ -314,9 +314,9 @@
          {
             // set a new session cookie
             Cookie newCookie = new Cookie(Globals.SESSION_COOKIE_NAME, sessionId);
-            if (log_.isDebugEnabled())
+            if (log_.isTraceEnabled())
             {
-               log_.debug("Setting cookie with session id:" + sessionId + " & name:" + Globals.SESSION_COOKIE_NAME);
+               log_.trace("Setting cookie with session id:" + sessionId + " & name:" + Globals.SESSION_COOKIE_NAME);
             }
 
             String contextPath = null;
@@ -882,9 +882,9 @@
 
       // Get all sessions
       Session sessions[] = findSessions();
-      if (log_.isDebugEnabled())
+      if (log_.isTraceEnabled())
       {
-         log_.debug("Looking for sessions that have expired ...");
+         log_.trace("Looking for sessions that have expired ...");
       }
 
       for (int i = 0; i < sessions.length; ++i)