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

Viraj Jasani updated HBASE-24396:
---------------------------------
    Description: RetryCounter utility which is used at multiple places like 
MetaRegionLocationCache, RecoverableZookeeper etc provides a method to sleep 
for a back off time as supplied by the backoff policy. However, sleep used by 
sleepUntilNextRetry() makes thread sleep and throws InterruptedException which 
clients have to deal with. Clients using sleep for any backoff policy should 
not worry about handling InterruptedException. Dealing with 
InterruptedException is something for sleepUntilNextRetry() to take care of.  
(was: RetryCounter utility which is used at multiple places like 
MetaRegionLocationCache, RecoverableZookeeper etc provides a method to sleep 
for a back off time as supplied by the backoff policy. However, sleep used by 
sleepUntilNextRetry() makes thread sleep and throws InterruptedException which 
clients have to deal with. Clients using sleep for exponential backoff should 
not worry about handling InterruptedException. Dealing with 
InterruptedException is something for sleepUntilNextRetry() to take care of.)

> RetryCounter#sleepUntilNextRetry should use uninterrupted sleep
> ---------------------------------------------------------------
>
>                 Key: HBASE-24396
>                 URL: https://issues.apache.org/jira/browse/HBASE-24396
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Viraj Jasani
>            Assignee: Viraj Jasani
>            Priority: Major
>
> RetryCounter utility which is used at multiple places like 
> MetaRegionLocationCache, RecoverableZookeeper etc provides a method to sleep 
> for a back off time as supplied by the backoff policy. However, sleep used by 
> sleepUntilNextRetry() makes thread sleep and throws InterruptedException 
> which clients have to deal with. Clients using sleep for any backoff policy 
> should not worry about handling InterruptedException. Dealing with 
> InterruptedException is something for sleepUntilNextRetry() to take care of.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to