[HBASE-6267] hbase.store.delete.expired.storefile should be true by default

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.94.1, 0.95.2
• Fix Version/s: 0.94.1, 0.95.0
• Component/s: regionserver
• Assignee: Andrew Kyle Purtell
• Reporter: Andrew Kyle Purtell
• Created: 25/Jun/12 17:14
• Updated: 26/Feb/13 08:16
• Resolved: 26/Jun/12 04:26

Description

HBASE-5199 introduces this logic into Store:

+      // Delete the expired store files before the compaction selection.
+      if (conf.getBoolean("hbase.store.delete.expired.storefile", false)
+          && (ttl != Long.MAX_VALUE) && (this.scanInfo.minVersions == 0)) {
+        CompactSelection expiredSelection = compactSelection
+            .selectExpiredStoreFilesToCompact(
+                EnvironmentEdgeManager.currentTimeMillis() - this.ttl);
+
+        // If there is any expired store files, delete them  by compaction.
+        if (expiredSelection != null) {
+          return expiredSelection;
+        }
+      }

Is there any reason why that should not be default true?