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/04/20 18:24:56 UTC

svn commit: r1740168 - /jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java

Author: mduerig
Date: Wed Apr 20 16:24:56 2016
New Revision: 1740168

URL: http://svn.apache.org/viewvc?rev=1740168&view=rev
Log:
OAK-3348: Cross gc sessions might introduce references to pre-compacted segments
Log a warning when a tar file is recovered due to a missing segment graph

Modified:
    jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java

Modified: jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java?rev=1740168&r1=1740167&r2=1740168&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java (original)
+++ jackrabbit/oak/trunk/oak-segment-next/src/main/java/org/apache/jackrabbit/oak/segment/file/TarReader.java Wed Apr 20 16:24:56 2016
@@ -798,6 +798,9 @@ class TarReader implements Closeable {
                     access.length() - size, access.length(), name);
             return this;
         }
+        if (!hasGraph()) {
+            log.warn("Recovering {}, which is missing its graph.", name);
+        }
 
         int pos = name.length() - "a.tar".length();
         char generation = name.charAt(pos);