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 mr...@apache.org on 2016/02/08 14:15:21 UTC

svn commit: r1729165 - in /jackrabbit/oak/trunk/oak-core/src: main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java

Author: mreutegg
Date: Mon Feb  8 13:15:21 2016
New Revision: 1729165

URL: http://svn.apache.org/viewvc?rev=1729165&view=rev
Log:
OAK-3995: Incorrect translation of old checkpoint format

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java?rev=1729165&r1=1729164&r2=1729165&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java Mon Feb  8 13:15:21 2016
@@ -228,7 +228,7 @@ class Checkpoints {
         for (Revision r : head) {
             int cId = r.getClusterId();
             if (cId == checkpoint.getClusterId()) {
-                revs.put(cId, r);
+                revs.put(cId, checkpoint);
             } else {
                 revs.put(cId, new Revision(checkpoint.getTimestamp(), 0, cId));
             }

Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java?rev=1729165&r1=1729164&r2=1729165&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/CheckpointsTest.java Mon Feb  8 13:15:21 2016
@@ -29,7 +29,6 @@ import org.apache.jackrabbit.oak.spi.sta
 import org.apache.jackrabbit.oak.spi.state.NodeState;
 import org.apache.jackrabbit.oak.stats.Clock;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 
@@ -353,7 +352,6 @@ public class CheckpointsTest {
         assertFalse(root.hasChildNode("baz"));
     }
 
-    @Ignore("OAK-3995")
     @Test
     public void sameClusterReadOldCheckpoint() throws Exception {
         DocumentStore store = new MemoryDocumentStore();