You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ja...@apache.org on 2008/05/29 17:09:14 UTC

svn commit: r661346 - /geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java

Author: jawarner
Date: Thu May 29 08:09:13 2008
New Revision: 661346

URL: http://svn.apache.org/viewvc?rev=661346&view=rev
Log:
GERONIMO-4087: Improve usability of gshell commands deploy/* when failing to connect to server

Modified:
    geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java

Modified: geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java?rev=661346&r1=661345&r2=661346&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java Thu May 29 08:09:13 2008
@@ -178,10 +178,14 @@
             }
             return manager;
         } catch (IOException e) {
-            log.error("caught ", e);
+        	if (log.isDebugEnabled()) {
+                log.debug("caught ", e);
+        	}
             DeploymentManagerCreationException deploymentManagerCreationException = 
                     (DeploymentManagerCreationException) new DeploymentManagerCreationException(e.getMessage()).initCause(e);
-            log.error("throwing ", deploymentManagerCreationException);
+            if (log.isDebugEnabled()) {
+                log.debug("throwing ", deploymentManagerCreationException);
+            }
             throw deploymentManagerCreationException;
         } catch (SecurityException e) {
             if (log.isDebugEnabled()) {