Author: reschke
Date: Tue Jan 17 13:51:29 2017
New Revision: 1779193

URL: http://svn.apache.org/viewvc?rev=1779193&view=rev
Log:
OAK-5461: backing out change due to test hangup on Windows

Modified:
    
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/BlobIdTracker.java
    
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DataStoreTrackerGCTest.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/BlobIdTracker.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/BlobIdTracker.java?rev=1779193&r1=1779192&r2=1779193&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/BlobIdTracker.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/BlobIdTracker.java
 Tue Jan 17 13:51:29 2017
@@ -443,12 +443,6 @@ public class BlobIdTracker implements Cl
          * @throws IOException
          */
         protected void removeRecords(File recs) throws IOException {
-            // Create a new generation writer.
-            // Merge the generations also before removing so that any id to be 
removed and also
-            // tracked in the generations are not resurrected
-            nextGeneration();
-            merge(generations, false);
-
             refLock.lock();
             try {
                 sort(getBlobRecordsFile());

Modified: 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DataStoreTrackerGCTest.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DataStoreTrackerGCTest.java?rev=1779193&r1=1779192&r2=1779193&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DataStoreTrackerGCTest.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DataStoreTrackerGCTest.java
 Tue Jan 17 13:51:29 2017
@@ -143,13 +143,6 @@ public class DataStoreTrackerGCTest {
         assertEquals(state.blobsPresent, existingAfterGC);
         // Tracked blobs should reflect deletions after gc
         assertEquals(state.blobsPresent, retrieveTracked(tracker));
-
-        // Create a snapshot
-        ScheduledFuture<?> scheduledFuture = newSingleThreadScheduledExecutor()
-            .schedule(tracker.new SnapshotJob(), 0, MILLISECONDS);
-        scheduledFuture.get();
-        // Tracked blobs should reflect deletions after gc and the deleted 
should not get resurrected
-        assertEquals(state.blobsPresent, retrieveTracked(tracker));
     }
 
     private HashSet<String> addNodeSpecialChars(DocumentNodeStore ds) throws 
Exception {
@@ -213,13 +206,6 @@ public class DataStoreTrackerGCTest {
         assertEquals(state.blobsPresent, existingAfterGC);
         // Tracked blobs should reflect deletions after gc and also the 
additions after
         assertEquals(state.blobsPresent, retrieveTracked(tracker));
-
-        // Create a snapshot
-        scheduledFuture = newSingleThreadScheduledExecutor()
-            .schedule(tracker.new SnapshotJob(), 0, MILLISECONDS);
-        scheduledFuture.get();
-        // Tracked blobs should reflect deletions after gc and the deleted 
should not get resurrected
-        assertEquals(state.blobsPresent, retrieveTracked(tracker));
     }
 
     @Test


Reply via email to