[jboss-cvs] JBoss Messaging SVN: r3262 - in trunk/src/main/org/jboss/jms/client: container and 2 other directories.

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

Author: timfox
Date: 2007-10-30 05:09:55 -0400 (Tue, 30 Oct 2007)
New Revision: 3262

Modified:
   trunk/src/main/org/jboss/jms/client/FailoverValve.java
   trunk/src/main/org/jboss/jms/client/JBossConnectionConsumer.java
   trunk/src/main/org/jboss/jms/client/container/ClientConsumer.java
   trunk/src/main/org/jboss/jms/client/container/ClusteringAspect.java
   trunk/src/main/org/jboss/jms/client/delegate/ClientClusteredConnectionFactoryDelegate.java
   trunk/src/main/org/jboss/jms/client/remoting/CallbackManager.java
Log:
Improved logging


Modified: trunk/src/main/org/jboss/jms/client/FailoverValve.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/FailoverValve.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/FailoverValve.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -174,7 +174,7 @@
 
          if (!acquired)
          {
-            log.warn(this + " could not close, trying again ...", new Exception());
+            log.debug(this + " could not close, trying again ...", new Exception());
             if (trace) { log.trace(debugValve()); }
          }
       }

Modified: trunk/src/main/org/jboss/jms/client/JBossConnectionConsumer.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/JBossConnectionConsumer.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/JBossConnectionConsumer.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -305,7 +305,7 @@
       }
       catch (Throwable t)
       {
-         log.warn("Connection consumer closing due to error in listening thread " + this, t);
+         log.debug("Connection consumer closing due to error in listening thread " + this, t);
          
          try
          {

Modified: trunk/src/main/org/jboss/jms/client/container/ClientConsumer.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/container/ClientConsumer.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/container/ClientConsumer.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -701,8 +701,7 @@
          if (trace) { log.trace(this + " got Closer result"); }
       }
       catch (InterruptedException e)
-      {
-         log.warn("Thread interrupted", e);
+      {         
       }
    }
 
@@ -713,8 +712,7 @@
          this.sessionExecutor.execute(runner);
       }
       catch (InterruptedException e)
-      {
-         log.warn("Thread interrupted", e);
+      {         
       }
    }
    

Modified: trunk/src/main/org/jboss/jms/client/container/ClusteringAspect.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/container/ClusteringAspect.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/container/ClusteringAspect.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -191,9 +191,9 @@
 
                   // In this case we should try back on the original server
 
-                  log.warn("Client attempted failover, but no failover attempt " +
-                           "has been detected on the server side. We will now try again on the original server " +
-                           "in case there was a temporary glitch on the client--server network");
+                  log.debug("Client attempted failover, but no failover attempt " +
+                            "has been detected on the server side. We will now try again on the original server " +
+                            "in case there was a temporary glitch on the client--server network");
 
                   delegate = getDelegateForNode(failedNodeID.intValue());
 

Modified: trunk/src/main/org/jboss/jms/client/delegate/ClientClusteredConnectionFactoryDelegate.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/delegate/ClientClusteredConnectionFactoryDelegate.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/delegate/ClientClusteredConnectionFactoryDelegate.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -109,7 +109,7 @@
          }
          catch (Throwable e)
          {
-            log.warn("Server communication to server[" + server + "] (" +
+            log.debug("Server communication to server[" + server + "] (" +
                delegates[server].getServerLocatorURI() + ") during establishCallback was broken, " +
                "trying the next one", e);
             if (remoting != null)
@@ -169,7 +169,7 @@
          }
          catch (Throwable warn)
          {
-            log.warn(warn, warn);
+            log.debug(warn, warn);
          }
 
          try
@@ -243,7 +243,7 @@
          }
          catch (MessagingNetworkFailureException e)
          {
-            log.warn("Server" + server + " was broken, loading AOP from next delegate", e);
+            log.debug("Server" + server + " was broken, loading AOP from next delegate", e);
          }
       }
 

Modified: trunk/src/main/org/jboss/jms/client/remoting/CallbackManager.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/remoting/CallbackManager.java	2007-10-30 04:25:11 UTC (rev 3261)
+++ trunk/src/main/org/jboss/jms/client/remoting/CallbackManager.java	2007-10-30 09:09:55 UTC (rev 3262)
@@ -112,7 +112,6 @@
       }
       else if (parameter instanceof ConnectionFactoryUpdate)
       {
-
          if (connectionfactoryCallbackHandler == null)
          {
             log.warn("ConnectionFactoryUpdate was received but there is no callbackHandler set");