[exo-jcr-commits] exo-jcr SVN: r828 - in jcr/branches/1.12.0-JBC/component/core/src: test/resources/conf/standalone and 1 other directory.

Author: pnedonosko Date: 2009-11-24 08:50:27 -0500 (Tue, 24 Nov 2009) New Revision: 828 Modified: jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JBossCacheStorageConnection.java jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JDBCCacheLoader.java jcr/branches/1.12.0-JBC/component/core/src/test/resources/conf/standalone/test-configuration.xml Log: EXOJCR-199: LOG warn to debug; javadocs Modified: jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JBossCacheStorageConnection.java =================================================================== --- jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JBossCacheStorageConnection.java 2009-11-24 13:22:15 UTC (rev 827) +++ jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JBossCacheStorageConnection.java 2009-11-24 13:50:27 UTC (rev 828) @@ -701,7 +701,7 @@ childNode.put(ITEM_ID, data.getIdentifier()); // 4. update all child nodes - updateTreePath(node, data.getQPath());//treePrint(node) + updateTreePath(node, data.getQPath()); //treePrint(node) // TODO cleanup // for (Node<Serializable, Object> child : node.getChildren()) @@ -737,7 +737,12 @@ // } } - //Update NodeData after update mixin + /** + * Update NodeData after update mixin. Used for JBossCache persistence only. + * + * @param data NodeData + * @throws RepositoryException if error occurs. + */ public void updateMixin(NodeData data) throws RepositoryException { startBatch(); Modified: jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JDBCCacheLoader.java =================================================================== --- jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JDBCCacheLoader.java 2009-11-24 13:22:15 UTC (rev 827) +++ jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/storage/jbosscache/JDBCCacheLoader.java 2009-11-24 13:50:27 UTC (rev 828) @@ -107,7 +107,10 @@ switch (m.getType()) { case PUT_DATA : - //LOG.warn("PUT_DATA modification"); + if (LOG.isDebugEnabled()) + { + LOG.debug("PUT_DATA modification"); + } // Process references // if (m.getFqn().get(0).equals(JBossCacheStorage.REFS)) // { @@ -115,7 +118,10 @@ // } break; case PUT_DATA_ERASE : - LOG.warn("PUT_DATA_ERASE modification"); + if (LOG.isDebugEnabled()) + { + LOG.debug("PUT_DATA_ERASE modification"); + } break; case PUT_KEY_VALUE : if (m.getFqn().size() == 2) @@ -143,10 +149,16 @@ break; case REMOVE_DATA : - LOG.warn("REMOVE_DATA modification"); + if (LOG.isDebugEnabled()) + { + LOG.debug("REMOVE_DATA modification"); + } break; case REMOVE_KEY_VALUE : - LOG.warn("REMOVE_KEY_VALUE modification"); + if (LOG.isDebugEnabled()) + { + LOG.debug("REMOVE_KEY_VALUE modification"); + } break; case REMOVE_NODE : @@ -161,7 +173,10 @@ break; case MOVE : - LOG.warn("MOVE modification"); + if (LOG.isDebugEnabled()) + { + LOG.debug("MOVE modification"); + } break; default : throw new CacheException("Unknown modification " + m.getType()); @@ -300,9 +315,6 @@ // time whether node is moved or just updated. This // solution is used as the fastest among other. conn.rename((NodeData)item); - - //update node data - //conn.update((NodeData)item); } else { @@ -449,7 +461,10 @@ else { attrs = null; - LOG.warn("Unexpected Fqn asked " + name); + if (LOG.isDebugEnabled()) + { + LOG.warn("Unexpected Fqn asked " + name); + } } } else if (name.get(0).equals(Fqn.ROOT) || name.get(0).equals(JBossCacheStorage.PROPS) @@ -574,7 +589,10 @@ else { exists = false; - LOG.warn("Unexpected Fqn asked " + name); + if (LOG.isDebugEnabled()) + { + LOG.debug("Unexpected Fqn asked " + name); + } } } else if (name.get(0).equals(Fqn.ROOT) || name.get(0).equals(JBossCacheStorage.PROPS) Modified: jcr/branches/1.12.0-JBC/component/core/src/test/resources/conf/standalone/test-configuration.xml =================================================================== --- jcr/branches/1.12.0-JBC/component/core/src/test/resources/conf/standalone/test-configuration.xml 2009-11-24 13:22:15 UTC (rev 827) +++ jcr/branches/1.12.0-JBC/component/core/src/test/resources/conf/standalone/test-configuration.xml 2009-11-24 13:50:27 UTC (rev 828) @@ -54,7 +54,8 @@ <property name="log4j.appender.file.layout.ConversionPattern" value="%d{dd.MM.yyyy HH:mm:ss} *%-5p* [%t] %c{1}: %m (%F, line %L) %n"/> - <property name="log4j.category.jcr.JBossCacheStorageConnection" value="DEBUG"/> + <property name="log4j.category.jcr.JBossCacheStorageConnection1" value="DEBUG"/> + <!-- property name="log4j.category.jcr.NodeImpl" value="DEBUG"/ --> </properties-param>