[jbosscache-commits] JBoss Cache SVN: r7052 - pojo/trunk/src/main/java/org/jboss/cache/pojo/impl.

Author: bstansberry(a)jboss.com Date: 2008-11-02 13:13:25 -0500 (Sun, 02 Nov 2008) New Revision: 7052 Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/AdvisedPojoHandler.java pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/CollectionClassHandler.java pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/InternalHelper.java pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/ObjectGraphHandler.java pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/PojoCacheDelegate.java pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/SerializableObjectHandler.java Log: Convert per-request DEBUG logging to TRACE Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/AdvisedPojoHandler.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/AdvisedPojoHandler.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/AdvisedPojoHandler.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -195,9 +195,9 @@ } // Need to make sure this is behind put such that obj.toString is done correctly. - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("internalPut(): inserting with fqn: " + fqn); + log.trace("internalPut(): inserting with fqn: " + fqn); } } @@ -236,9 +236,9 @@ // Remember to remove the interceptor from in-memory object but make sure it belongs to me first. if (interceptor != null) { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("regularRemoveObject(): removed cache interceptor fqn: " + fqn + " interceptor: " + interceptor); + log.trace("regularRemoveObject(): removed cache interceptor fqn: " + fqn + " interceptor: " + interceptor); } util_.detachInterceptor(advisor, interceptor); } Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/CollectionClassHandler.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/CollectionClassHandler.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/CollectionClassHandler.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -107,9 +107,9 @@ if (obj instanceof Map) { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("collectionPutObject(): aspectized obj is a Map type of size: " + ((Map) obj).size()); + log.trace("collectionPutObject(): aspectized obj is a Map type of size: " + ((Map) obj).size()); } // Let's replace it with a proxy if necessary @@ -140,9 +140,9 @@ } else if (obj instanceof List) { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("collectionPutObject(): aspectized obj is a List type of size: " + log.trace("collectionPutObject(): aspectized obj is a List type of size: " + ((List) obj).size()); } @@ -174,9 +174,9 @@ } else if (obj instanceof Set) { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("collectionPutObject(): aspectized obj is a Set type of size: " + log.trace("collectionPutObject(): aspectized obj is a Set type of size: " + ((Set) obj).size()); } Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/InternalHelper.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/InternalHelper.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/InternalHelper.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -329,8 +329,8 @@ { final int RETRY = 5; - if (log.isDebugEnabled()) - log.debug("lockPojo(): id:" + id); + if (log.isTraceEnabled()) + log.trace("lockPojo(): id:" + id); boolean isNeeded = true; int retry = 0; Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/ObjectGraphHandler.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/ObjectGraphHandler.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/ObjectGraphHandler.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -84,9 +84,9 @@ protected Object remove(Fqn<?> fqn, Reference reference, Object pojo) throws CacheException { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("remove(): removing object fqn: " + reference + log.trace("remove(): removing object fqn: " + reference + " Will just de-reference it."); } removeFromReference(fqn, reference); Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/PojoCacheDelegate.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/PojoCacheDelegate.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/PojoCacheDelegate.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -68,9 +68,9 @@ if (pojo != null) { // we already have an advised instance - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("getObject(): id: " + fqn + " retrieved from existing instance directly. "); + log.trace("getObject(): id: " + fqn + " retrieved from existing instance directly. "); } return pojo; } @@ -105,9 +105,9 @@ boolean allowArray = source instanceof ArrayInterceptable; if (oldValue == obj && skipDuplicateAttach(obj, allowArray)) { - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("putObject(): id: " + fqn + " pojo is already in the cache. Return right away."); + log.trace("putObject(): id: " + fqn + " pojo is already in the cache. Return right away."); } return obj; } @@ -132,8 +132,8 @@ else { internalFqn = createInternalFqn(fqn, obj); - if (log.isDebugEnabled()) - log.debug("attach(): id: " + fqn + " will store the pojo in the internal area: " + internalFqn); + if (log.isTraceEnabled()) + log.trace("attach(): id: " + fqn + " will store the pojo in the internal area: " + internalFqn); handler.put(internalFqn, reference, obj); @@ -186,9 +186,9 @@ // store PojoReference pojoReference.setFqn(internalFqn); internal_.putPojoReference(fqn, pojoReference, field); - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("put(): inserting PojoReference with id: " + fqn); + log.trace("put(): inserting PojoReference with id: " + fqn); } // store obj in the internal fqn return internalFqn; @@ -203,7 +203,7 @@ Node base = cache.getRoot().getChild(f); if (base == null) { - log.debug("The node retrieved is null from fqn: " + f); + log.trace("The node retrieved is null from fqn: " + f); return; } base.addChild(child); @@ -236,9 +236,9 @@ - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("removeObject(): removing object from id: " + fqn + log.trace("removeObject(): removing object from id: " + fqn + " with the corresponding internal id: " + internalFqn); } @@ -278,9 +278,9 @@ } findChildObjects(fqn, map); - if (log.isDebugEnabled()) + if (log.isTraceEnabled()) { - log.debug("_findObjects(): id: " + fqn + " size of pojos found: " + map.size()); + log.trace("_findObjects(): id: " + fqn + " size of pojos found: " + map.size()); } return map; } @@ -298,8 +298,8 @@ return null; } - if (log.isDebugEnabled()) - log.debug("getObject(): id: " + fqn + " with a corresponding internal id: " + internalFqn); + if (log.isTraceEnabled()) + log.trace("getObject(): id: " + fqn + " with a corresponding internal id: " + internalFqn); /** * Reconstruct the managed POJO Modified: pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/SerializableObjectHandler.java =================================================================== --- pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/SerializableObjectHandler.java 2008-11-02 17:47:24 UTC (rev 7051) +++ pojo/trunk/src/main/java/org/jboss/cache/pojo/impl/SerializableObjectHandler.java 2008-11-02 18:13:25 UTC (rev 7052) @@ -62,9 +62,9 @@ protected void put(Fqn<?> fqn, Reference reference, Object obj) throws CacheException { // Note that JBoss Serialization can serialize any type now. - if (log_.isDebugEnabled()) + if (log_.isTraceEnabled()) { - log_.debug("put(): obj (" + obj.getClass() + ") is non-advisable but serialize it anyway. " + log_.trace("put(): obj (" + obj.getClass() + ") is non-advisable but serialize it anyway. " + "Note that if it is non-serializable we require to use JBoss Serialization."); }