You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2013/10/03 21:18:19 UTC

svn commit: r1528978 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java

Author: jukka
Date: Thu Oct  3 19:18:19 2013
New Revision: 1528978

URL: http://svn.apache.org/r1528978
Log:
OAK-1031: SegmentMK: Fewer segment lookups

Drop caching of map records to reduce the concurrency overhead of the cache lookups

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java?rev=1528978&r1=1528977&r2=1528978&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/Segment.java Thu Oct  3 19:18:19 2013
@@ -239,13 +239,8 @@ public class Segment {
         }
     }
 
-    MapRecord readMap(final RecordId id) {
-        return store.getRecord(id, new Callable<MapRecord>() {
-            @Override
-            public MapRecord call() {
-                return getSegment(id).readMap(id.getOffset());
-            }
-        });
+    MapRecord readMap(RecordId id) {
+        return getSegment(id).readMap(id.getOffset());
     }
 
     MapRecord readMap(int offset) {