giving an appropriate name to the SpaceReclaimer
authorKiran Ayyagari <kayyagari@apache.org>
Wed, 20 May 2015 08:15:00 +0000 (08:15 +0000)
committerKiran Ayyagari <kayyagari@apache.org>
Wed, 20 May 2015 08:15:00 +0000 (08:15 +0000)
mavibot/src/main/java/org/apache/directory/mavibot/btree/PageReclaimer.java [moved from mavibot/src/main/java/org/apache/directory/mavibot/btree/SpaceReclaimer.java with 97% similarity]
mavibot/src/main/java/org/apache/directory/mavibot/btree/RecordManager.java
mavibot/src/test/java/org/apache/directory/mavibot/btree/PageReclaimerTest.java [moved from mavibot/src/test/java/org/apache/directory/mavibot/btree/SpaceReclaimerTest.java with 95% similarity]

@@ -35,23 +35,23 @@ import org.slf4j.LoggerFactory;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-public class SpaceReclaimer
+public class PageReclaimer
 {
     /** the record manager */
     private RecordManager rm;
 
     /** The LoggerFactory used by this class */
-    protected static final Logger LOG = LoggerFactory.getLogger( SpaceReclaimer.class );
+    protected static final Logger LOG = LoggerFactory.getLogger( PageReclaimer.class );
 
     /** a flag to detect the running state */
     private boolean running = false;
     
     /**
-     * Creates a new instance of SpaceReclaimer.
+     * Creates a new instance of PageReclaimer.
      *
      * @param rm the record manager
      */
-    public SpaceReclaimer( RecordManager rm )
+    public PageReclaimer( RecordManager rm )
     {
         this.rm = rm;
     }    
index be687fc..ebb27d4 100644 (file)
@@ -210,12 +210,12 @@ public class RecordManager extends AbstractTransactionManager
     private ReentrantLock freePageLock = new ReentrantLock();
 
     /** the space reclaimer */
-    private SpaceReclaimer reclaimer;
+    private PageReclaimer reclaimer;
 
     /** variable to keep track of the write commit count */
     private int commitCount = 0;
 
-    /** the threshold at which the SpaceReclaimer will be run to free the copied pages */
+    /** the threshold at which the PageReclaimer will be run to free the copied pages */
     // FIXME the below value is derived after seeing that anything higher than that
     // is resulting in a "This thread does not hold the transactionLock" error
     private int spaceReclaimerThreshold = 70;
@@ -293,7 +293,7 @@ public class RecordManager extends AbstractTransactionManager
                 loadRecordManager();
             }
 
-            reclaimer = new SpaceReclaimer( this );
+            reclaimer = new PageReclaimer( this );
             runReclaimer();
         }
         catch ( Exception e )
@@ -306,7 +306,7 @@ public class RecordManager extends AbstractTransactionManager
 
 
     /**
-     * runs the SpaceReclaimer to free the copied pages
+     * runs the PageReclaimer to free the copied pages
      */
     private void runReclaimer()
     {
@@ -323,7 +323,7 @@ public class RecordManager extends AbstractTransactionManager
         }
         catch ( Exception e )
         {
-            LOG.warn( "SpaceReclaimer failed to free the pages", e );
+            LOG.warn( "PageReclaimer failed to free the pages", e );
         }
     }
 
@@ -41,7 +41,7 @@ import org.junit.rules.TemporaryFolder;
  *\r
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>\r
  */\r
-public class SpaceReclaimerTest\r
+public class PageReclaimerTest\r
 {\r
     private static final String TREE_NAME = "uid-tree";\r
 \r
@@ -121,7 +121,7 @@ public class SpaceReclaimerTest
      * with the reclaimer threshold 10 and total entries of 1120\r
      * there was a condition that resulted in OOM while reopening the RM\r
      * \r
-     * This issue was fixed after SpaceReclaimer was updated to run in\r
+     * This issue was fixed after PageReclaimer was updated to run in\r
      * a transaction.\r
      * \r
      * This test is present to verify the fix\r