[
https://issues.apache.org/jira/browse/HBASE-10524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13905502#comment-13905502
]
Feng Honghua commented on HBASE-10524:
--------------------------------------
Yes, both taskLoop and run handle InterruptedException already(though some just
handle for its own sleep), letting getTaskList()/taskLoop() just throw out IE
and letting run() handle IE does make the code cleaner. Good to me:-)
btw: I made a new patch for HBASE-10521, would you help review it again?
thanks:-)
> Correct wrong handling and add proper handling for swallowed
> InterruptedException thrown by Thread.sleep in regionserver
> ------------------------------------------------------------------------------------------------------------------------
>
> Key: HBASE-10524
> URL: https://issues.apache.org/jira/browse/HBASE-10524
> Project: HBase
> Issue Type: Sub-task
> Components: regionserver
> Reporter: Feng Honghua
> Assignee: Feng Honghua
> Attachments: HBASE-10524-trunk_v1.patch, HBASE-10524-trunk_v2.patch,
> split.patch
>
>
> A sub-task of HBASE-10497
> # correct wrong handling of InterruptedException where
> Thread.currentThread.interrupt() is called within while loops
> # add proper handling for swallowed InterruptedException
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
