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 re...@apache.org on 2016/03/15 16:37:53 UTC
svn commit: r1735109 -
/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java
Author: reschke
Date: Tue Mar 15 15:37:53 2016
New Revision: 1735109
URL: http://svn.apache.org/viewvc?rev=1735109&view=rev
Log:
OAK-4125: fix potential timing issue in RDBBlobStoreTest
Modified:
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java
Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java?rev=1735109&r1=1735108&r2=1735109&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/blob/RDBBlobStoreTest.java Tue Mar 15 15:37:53 2016
@@ -182,9 +182,11 @@ public class RDBBlobStoreTest extends Ab
}
}
// Force update to update timestamp
+ long beforeUpdateTs = System.currentTimeMillis() - 100;
RDBBlobStoreFriend.storeBlock(blobStore, digest, 0, data);
// Metadata row should not have been touched
- Assert.assertFalse(blobStore.deleteChunks(ImmutableList.of(id), System.currentTimeMillis() - 100));
+ Assert.assertFalse("entry was cleaned although it shouldn't have",
+ blobStore.deleteChunks(ImmutableList.of(id), beforeUpdateTs));
// Actual data row should still be present
Assert.assertNotNull(RDBBlobStoreFriend.readBlockFromBackend(blobStore, digest));
}