You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2015/08/28 10:14:58 UTC

svn commit: r1698275 - /sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java

Author: cziegeler
Date: Fri Aug 28 08:14:58 2015
New Revision: 1698275

URL: http://svn.apache.org/r1698275
Log:
SLING-4972 : Debug log output of OakResourceListener is not usable

Modified:
    sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java

Modified: sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java?rev=1698275&r1=1698274&r2=1698275&view=diff
==============================================================================
--- sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java (original)
+++ sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/OakResourceListener.java Fri Aug 28 08:14:58 2015
@@ -47,7 +47,6 @@ import org.apache.jackrabbit.oak.spi.com
 import org.apache.sling.api.SlingConstants;
 import org.apache.sling.api.resource.Resource;
 import org.apache.sling.api.resource.ResourceResolver;
-import org.apache.sling.jcr.resource.internal.helper.jcr.JcrResourceProvider;
 import org.apache.sling.jcr.resource.internal.helper.jcr.PathMapper;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.Constants;
@@ -107,6 +106,7 @@ public class OakResourceListener extends
     /**
      * Dispose this listener.
      */
+    @Override
     public void close() throws IOException {
         serviceRegistration.unregister();
         this.support.dispose();
@@ -121,7 +121,9 @@ public class OakResourceListener extends
             final CommitInfo commitInfo) {
         final Map<String, Object> changes = toEventProperties(added, deleted, changed);
         addCommitInfo(changes, commitInfo);
-        logger.debug("added(changes={})", changes);
+        if ( logger.isDebugEnabled() ) {
+            logger.debug("added(path={}, added={}, deleted={}, changed={})", new Object[] {path, added, deleted, changed});
+        }
         sendOsgiEvent(path, TOPIC_RESOURCE_ADDED, changes, properties);
     }
 
@@ -134,7 +136,9 @@ public class OakResourceListener extends
             final CommitInfo commitInfo) {
         final Map<String, Object> changes = toEventProperties(added, deleted, changed);
         addCommitInfo(changes, commitInfo);
-        logger.debug("deleted(changes={})", changes);
+        if ( logger.isDebugEnabled() ) {
+            logger.debug("deleted(path={}, added={}, deleted={}, changed={})", new Object[] {path, added, deleted, changed});
+        }
         sendOsgiEvent(path, TOPIC_RESOURCE_REMOVED, changes, properties);
     }
 
@@ -147,7 +151,9 @@ public class OakResourceListener extends
             final CommitInfo commitInfo) {
         final Map<String, Object> changes = toEventProperties(added, deleted, changed);
         addCommitInfo(changes, commitInfo);
-        logger.debug("changed (changes={})", changes);
+        if ( logger.isDebugEnabled() ) {
+            logger.debug("changed(path={}, added={}, deleted={}, changed={})", new Object[] {path, added, deleted, changed});
+        }
         sendOsgiEvent(path, TOPIC_RESOURCE_CHANGED, changes, properties);
     }