svn commit: r1845584 - /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java

[frm]

Author: frm
Date: Fri Nov  2 13:13:19 2018
New Revision: 1845584

URL: http://svn.apache.org/viewvc?rev=1845584&view=rev
Log:
OAK-7877 - Avoid unnecessary operations when the read operations log is disabled

Modified:
    
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java

Modified: 
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java?rev=1845584&r1=1845583&r2=1845584&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/CachingSegmentReader.java
 Fri Nov  2 13:13:19 2018
@@ -142,7 +142,9 @@ public class CachingSegmentReader implem
     @NotNull
     @Override
     public SegmentNodeState readNode(@NotNull RecordId id) {
-        LOG.trace(System.currentTimeMillis() + " " + 
safeEncode(Thread.currentThread().getName()) + " n? " + id.toString());
+        if (LOG.isTraceEnabled()) {
+            LOG.trace("{} n? {}", Thread.currentThread().getId(), id);
+        }
         return new SegmentNodeState(this, writer, blobStore, id, readStats);
     }
 
@@ -154,9 +156,10 @@ public class CachingSegmentReader implem
 
     @NotNull
     @Override
-    public SegmentPropertyState readProperty(
-            @NotNull RecordId id, @NotNull PropertyTemplate template) {
-        LOG.trace(System.currentTimeMillis() + " " + 
safeEncode(Thread.currentThread().getName()) + " p? " + id.toString());
+    public SegmentPropertyState readProperty(@NotNull RecordId id, @NotNull 
PropertyTemplate template) {
+        if (LOG.isTraceEnabled()) {
+            LOG.trace("{} p? {}", Thread.currentThread().getId(), id);
+        }
         return new SegmentPropertyState(this, id, template);
     }