[jira] [Reopened] (HBASE-19616) Review of LogCleaner Class

["stack (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-19616?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack reopened HBASE-19616:
---------------------------

Reopening to apply to branch-2.

Thanks for the detail [~belugabehr]

> Review of LogCleaner Class
> --------------------------
>
>                 Key: HBASE-19616
>                 URL: https://issues.apache.org/jira/browse/HBASE-19616
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 3.0.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Minor
>             Fix For: 3.0.0
>
>         Attachments: HBASE-19616.1.patch, HBASE-19616.2.patch, HBASE-19616.3.patch, HBASE-19616.branch-2.patch
>
>
> * Parameterize logging
> * Remove compiler-reported dead code to re-enabling useful logging
> * Use ArrayList instead of LinkedList
> Used a CountDownLatch to replace a bunch of the existing code. It currently loops with a 500ms interval to check if some sort of condition has been met until the amount of time spent looping is greater than some timeout value. Using a CountDownLatch allows one or more threads to wait until a set of operations being performed in other threads completes. It will not blindly sleep between checks and it will return immediately after the condition is met. This removes the HBase configuration that controls the sleep interval.
>  
> I also cleaned up the unit tests a bit and enhanced the logging of this class to ease troubleshooting.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)