svn commit: r1594932 - in /tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config: AutoConfig.java ConfigurationFactory.java

[jlmonteiro]

Author: jlmonteiro
Date: Thu May 15 14:44:05 2014
New Revision: 1594932

URL: http://svn.apache.org/r1594932
Log:
Wrapping the debug log with isDebugEnabled

Modified:
    
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
    
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java

Modified: 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
URL: 
http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java?rev=1594932&r1=1594931&r2=1594932&view=diff
==============================================================================
--- 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
 (original)
+++ 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
 Thu May 15 14:44:05 2014
@@ -2170,21 +2170,29 @@ public class AutoConfig implements Dynam
 
         public void dump() {
             for (final String s : resourceAdapterIds) {
-                logger.debug(appId + " module contains resource adapter id: " 
+ s);
+                if (logger.isDebugEnabled()) {
+                    logger.debug(appId + " module contains resource adapter 
id: " + s);
+                }
             }
             for (final String s : resourceIdsByType.keySet()) {
                 for (final String value : resourceIdsByType.get(s)) {
-                    logger.debug(appId + " module contains resource type: " + 
s + " --> " + value);
+                    if (logger.isDebugEnabled()) {
+                        logger.debug(appId + " module contains resource type: 
" + s + " --> " + value);
+                    }
                 }
             }
             for (final String s : resourceEnvIdsByType.keySet()) {
                 for (final String value : resourceEnvIdsByType.get(s)) {
-                    logger.debug(appId + " module contains resource env type: 
" + s + " --> " + value);
+                    if (logger.isDebugEnabled()) {
+                        logger.debug(appId + " module contains resource env 
type: " + s + " --> " + value);
+                    }
                 }
             }
             for (final String s : containerIdsByType.keySet()) {
                 for (final String value : containerIdsByType.get(s)) {
-                    logger.debug(appId + " module contains container type: " + 
s + " --> " + value);
+                    if (logger.isDebugEnabled()) {
+                        logger.debug(appId + " module contains container type: 
" + s + " --> " + value);
+                    }
                 }
             }
         }

Modified: 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
URL: 
http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java?rev=1594932&r1=1594931&r2=1594932&view=diff
==============================================================================
--- 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
 (original)
+++ 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
 Thu May 15 14:44:05 2014
@@ -1428,7 +1428,9 @@ public class ConfigurationFactory implem
         final OpenEjbConfiguration runningConfig = getRunningConfig();
         if (runningConfig != null) {
             for (final ResourceInfo resourceInfo : 
runningConfig.facilities.resources) {
-                logger.debug(String.format("Trying to match resource type %s 
with classname %s, service %s.", type, resourceInfo.className, 
resourceInfo.service));
+                if (logger.isDebugEnabled()) {
+                    logger.debug(String.format("Trying to match resource type 
%s with classname %s, service %s.", type, resourceInfo.className, 
resourceInfo.service));
+                }
                 if ((type != null && type.equals(resourceInfo.className) || 
isResourceType(resourceInfo.service, resourceInfo.types, type)) && 
implies(required, resourceInfo.properties)) {
                     resourceIds.add(resourceInfo.id);
                     resourceIds.addAll(resourceInfo.aliases);