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

Tak Lon (Stephen) Wu edited comment on HBASE-20401 at 7/20/18 11:39 PM:
------------------------------------------------------------------------

[~reidchan] for branch-1 patch, can we wait till HBASE-20559 get in first? 
because the dynamic configurations checking need to be added in this branch-1's 
patch (then we don't need another JIRA, I think it could be done on Monday if 
[~zyork] reviews HBASE-20559 ;P )

meanwhile, for branch-2 and master, it's safe to commit.


was (Author: taklwu):
[~reidchan] for branch-1 patch, can we wait till HBASE-20559 get it first? 
because the dynamic configuration checking need to be added in this branch-1 
patch (then we don't need another JIRA, I think it could be done on Monday if 
[~zyork] reviews HBASE-20559 ;P ) 

meanwhile, for branch-2 and master, it's safe to commit.

> Make `MAX_WAIT` and `waitIfNotFinished` in CleanerContext configurable
> ----------------------------------------------------------------------
>
>                 Key: HBASE-20401
>                 URL: https://issues.apache.org/jira/browse/HBASE-20401
>             Project: HBase
>          Issue Type: Improvement
>          Components: master
>    Affects Versions: 3.0.0, 1.5.0, 2.0.0-beta-1, 1.4.4, 2.0.0
>            Reporter: Tak Lon (Stephen) Wu
>            Assignee: Tak Lon (Stephen) Wu
>            Priority: Minor
>              Labels: beginner
>         Attachments: HBASE-20401.branch-1.001.patch, 
> HBASE-20401.master.001.patch, HBASE-20401.master.002.patch, 
> HBASE-20401.master.003.patch, HBASE-20401.master.004.patch, 
> HBASE-20401.master.005.patch, HBASE-20401.master.006.patch
>
>
> When backporting HBASE-18309 in HBASE-20352, the deleteFiles calls 
> CleanerContext.java#getResult with a waitIfNotFinished timeout to wait for 
> notification (notify) from the fs.delete file thread. there might be two 
> situation need to tune the MAX_WAIT in CleanerContext or waitIfNotFinished 
> when LogClearner call getResult.
>  # fs.delete never complete (strange but possible), then we need to wait for 
> a max of 60 seconds. here, 60 seconds might be too long
>  # getResult is waiting in the period of 500 milliseconds, but the fs.delete 
> has completed and setFromClear is set but yet notify(). one might want to 
> tune this 500 milliseconds to 200 or less .



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

Reply via email to