You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by jl...@apache.org on 2014/05/15 16:44:06 UTC
svn commit: r1594932 - in
/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config:
AutoConfig.java ConfigurationFactory.java
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);