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 md...@apache.org on 2016/05/24 07:31:08 UTC

svn commit: r1745313 - /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/http/HttpStore.java

Author: mduerig
Date: Tue May 24 07:31:08 2016
New Revision: 1745313

URL: http://svn.apache.org/viewvc?rev=1745313&view=rev
Log:
OAK-4396: HttpStore.containsSegment throws SNFE instead of returning false for non existing segments
FIXME

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

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/http/HttpStore.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/http/HttpStore.java?rev=1745313&r1=1745312&r2=1745313&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/http/HttpStore.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/http/HttpStore.java Tue May 24 07:31:08 2016
@@ -134,6 +134,7 @@ public class HttpStore implements Segmen
     }
 
     @Override
+    // FIXME OAK-4396: HttpStore.containsSegment throws SNFE instead of returning false for non existing segments
     public boolean containsSegment(SegmentId id) {
         return id.sameStore(this) || readSegment(id) != null;
     }