Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-6267

hbase.store.delete.expired.storefile should be true by default

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.94.1, 0.95.2
    • 0.94.1, 0.95.0
    • regionserver
    • None

    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?

      Attachments

        1. HBASE-6267-0.94.patch
          3 kB
          Andrew Kyle Purtell
        2. HBASE-6267.patch
          3 kB
          Andrew Kyle Purtell

        Activity

          People

            apurtell Andrew Kyle Purtell
            apurtell Andrew Kyle Purtell
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: