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/03/22 17:45:56 UTC

svn commit: r1736211 - /jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java

Author: mduerig
Date: Tue Mar 22 16:45:56 2016
New Revision: 1736211

URL: http://svn.apache.org/viewvc?rev=1736211&view=rev
Log:
OAK-3972: Forward edges missing in SegmentGraph
Include forward references reachable from initial roots

Modified:
    jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java

Modified: jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java?rev=1736211&r1=1736210&r2=1736211&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java (original)
+++ jackrabbit/oak/trunk/oak-segment/src/main/java/org/apache/jackrabbit/oak/plugins/segment/file/FileStore.java Tue Mar 22 16:45:56 2016
@@ -1460,7 +1460,10 @@ public class FileStore implements Segmen
         public void traverseSegmentGraph(
             @Nonnull Set<UUID> roots,
             @Nonnull SegmentGraphVisitor visitor) throws IOException {
-            for (TarReader reader : super.readers) {
+
+            List<TarReader> readers = super.readers;
+            super.includeForwardReferences(readers, roots);
+            for (TarReader reader : readers) {
                 reader.traverseSegmentGraph(checkNotNull(roots), checkNotNull(visitor));
             }
         }