Author: stefanegli
Date: Wed Feb  8 14:19:13 2017
New Revision: 1782179

URL: http://svn.apache.org/viewvc?rev=1782179&view=rev
Log:
OAK-5601 : more explanation for the testcase added

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

Modified: 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/JournalGCTest.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/JournalGCTest.java?rev=1782179&r1=1782178&r2=1782179&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/JournalGCTest.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/JournalGCTest.java
 Wed Feb  8 14:19:13 2017
@@ -139,10 +139,14 @@ public class JournalGCTest {
      * reproducing OAK-5601:
      * <ul>
      *  <li>have two documentMk's, one to make changes, one does only read</li>
-     *  <li>make a commit, let 2 hours pass, run gc, then read it from the 
other documentMk</li>
-     *  <li>the gc (1h timeout) will have cleaned up that 2h old journal 
entry, resulting in
+     *  <li>make a commit, let 1.2 seconds pass, run gc, then read it from the 
other documentMk</li>
+     *  <li>the gc (1sec timeout) will have cleaned up that 1.2sec old journal 
entry, resulting in
      *      a missing journal entry exception when reading from the 2nd 
documentMk</li>
      * </ul>
+     * What the test has to ensure is that the JournalEntry does the query, 
then blocks that
+     * thread to let the GC happen, then continues on with find(). This 
results in those
+     * revisions that the JournalEntry got back from the query to be removed 
and
+     * thus end up missing by later on in addTo.
      */
     @Test
     @Ignore("OAK-5601")


Reply via email to