geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xuhaih...@apache.org
Subject svn commit: r1060681 - /geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java
Date Wed, 19 Jan 2011 05:25:51 GMT
Author: xuhaihong
Date: Wed Jan 19 05:25:50 2011
New Revision: 1060681

URL: http://svn.apache.org/viewvc?rev=1060681&view=rev
Log:
No function change, just update some log level

Modified:
    geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java

Modified: geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java?rev=1060681&r1=1060680&r2=1060681&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java
(original)
+++ geronimo/server/trunk/plugins/bval/geronimo-bval/src/main/java/org/apache/geronimo/bval/ValidatorFactoryGBean.java
Wed Jan 19 05:25:50 2011
@@ -163,7 +163,7 @@ public class ValidatorFactoryGBean imple
                                 out.close();
                                 validationConfigURL = new URL("jar:"+tmpArchiveFile.toURI().toURL()+"!/"+validationConfig);
                             } catch (IOException e) {
-                                log.debug("Error processing validation configuration "+validationConfig+"
in "+moduleName + " Using default factory.", e);
+                                log.warn("Error processing validation configuration "+validationConfig+"
in "+moduleName + " Using default factory.", e);
                                 createDefaultFactory();
                                 return factory;
                             }
@@ -173,7 +173,7 @@ public class ValidatorFactoryGBean imple
                     JAXBElement<ValidationConfigType> root = unmarshaller.unmarshal(stream,
ValidationConfigType.class);
                     validationConfigType = root.getValue();
                 } catch(Throwable t) {
-                    log.debug("Unable to create module ValidatorFactory instance.  Using
default factory", t);
+                    log.warn("Unable to create module ValidatorFactory instance.  Using default
factory", t);
                     createDefaultFactory();
                     return factory;
                 }
@@ -267,12 +267,16 @@ public class ValidatorFactoryGBean imple
             }
         }
         for (PropertyType property : xmlConfig.getProperty()) {
-            log.debug("Found property '" + property.getName() + "' with value '" + property.getValue()
+ "' in " + validationConfig);
+            if (log.isDebugEnabled()) {
+                log.debug("Found property '" + property.getName() + "' with value '" + property.getValue()
+ "' in " + validationConfig);
+            }
             target.addProperty(property.getName(), property.getValue());
         }
         for (JAXBElement<String> mappingFileNameElement : xmlConfig.getConstraintMapping())
{
             String mappingFileName = mappingFileNameElement.getValue();
-            log.debug("Opening input stream for " + mappingFileName);
+            if (log.isDebugEnabled()) {
+                log.debug("Opening input stream for " + mappingFileName);
+            }
             InputStream in = null;
             try {
                 if(moduleName == null) {
@@ -307,7 +311,9 @@ public class ValidatorFactoryGBean imple
 
     public void doStop() throws Exception {
         factory = null;
-        log.debug("Stopped " + objectName);
+        if (log.isDebugEnabled()) {
+            log.debug("Stopped " + objectName);
+        }
     }
 
     public void doFail() {



Mime
View raw message