hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "HBase Review Board (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-2792) Create a better way to chain log cleaners
Date Fri, 23 Jul 2010 06:11:54 GMT

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

HBase Review Board commented on HBASE-2792:
-------------------------------------------

Message from: "Chongxin Li" <lichongxin@zju.edu.cn>

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/372/
-----------------------------------------------------------

(Updated 2010-07-22 23:00:05.908203)


Review request for hbase.


Changes
-------

HBASE-2792: Create a better way to chain log cleaners


Summary
-------

HBASE-2792: Create a better way to chain log cleaners


This addresses bug HBASE-2792.
    http://issues.apache.org/jira/browse/HBASE-2792


Diffs (updated)
-----

  src/main/java/org/apache/hadoop/hbase/master/LogCleanerDelegate.java 3ca3611 
  src/main/java/org/apache/hadoop/hbase/master/OldLogsCleaner.java 37b2c3c 
  src/main/java/org/apache/hadoop/hbase/replication/master/ReplicationLogCleaner.java eb859aa

  src/main/resources/hbase-default.xml e3a9669 
  src/test/java/org/apache/hadoop/hbase/master/TestOldLogsCleaner.java a92e0da 

Diff: http://review.hbase.org/r/372/diff


Testing
-------

Unit test TestOldLogsCleaner passed.


Thanks,

Chongxin




> Create a better way to chain log cleaners
> -----------------------------------------
>
>                 Key: HBASE-2792
>                 URL: https://issues.apache.org/jira/browse/HBASE-2792
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Jean-Daniel Cryans
>            Assignee: Li Chongxin
>             Fix For: 0.90.0
>
>
> From Stack's review of HBASE-2223:
> {quote}
> Why this implementation have to know about other implementations?  Can't we do a chain
of decision classes? Any class can say no?  As soon as any decision class says no, we exit
the chain.... So in this case, first on the chain would be the ttl decision... then would
be this one... and third would be the snapshotting decision. You don't have to do the chain
as part of this patch but please open an issue to implement.
> {quote}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message