hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-6267) hbase.store.delete.expired.storefile should be true by default
Date Tue, 26 Jun 2012 06:09:44 GMT

    [ https://issues.apache.org/jira/browse/HBASE-6267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13401194#comment-13401194
] 

Hudson commented on HBASE-6267:
-------------------------------

Integrated in HBase-0.94 #281 (See [https://builds.apache.org/job/HBase-0.94/281/])
    HBASE-6267. hbase.store.delete.expired.storefile should be true by default (Revision 1353813)

     Result = FAILURE
apurtell : 
Files : 
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
* /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/io/hfile/TestScannerSelectionUsingTTL.java

                
> hbase.store.delete.expired.storefile should be true by default
> --------------------------------------------------------------
>
>                 Key: HBASE-6267
>                 URL: https://issues.apache.org/jira/browse/HBASE-6267
>             Project: HBase
>          Issue Type: Improvement
>          Components: regionserver
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Andrew Purtell
>            Assignee: Andrew Purtell
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6267-0.94.patch, HBASE-6267.patch
>
>
> HBASE-5199 introduces this logic into Store:
> {code}
> +      // 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;
> +        }
> +      }
> {code}
> Is there any reason why that should not be default {{true}}?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message