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

Feng Honghua updated HBASE-10520:
---------------------------------

    Attachment: HBASE-10520-trunk_v3.patch

v3 attached which changes to rethrow InterruptedIOException rather than 
recording and restoring interrupt status in v2, and looks like it is cleaner 
and still reasonable:
# waitForServerUp(private) is called once by startup(), and startup() itself 
can throw IOException and InterruptedException.
# waitForServerDown(private) is called by shutdown() which itself throws 
IOException, by killCurrentActiveZooKeeperServer which throws IOException and 
InterruptedException, by killOneBackupZooKeeperServer which throws IOException 
and InterruptedException
# to align the interface, InterruptedIOException rather than 
InterruptedException is thrown, since shutdown() throws only IOException.

Ping [~nkeywal] and thanks :-)

> Add handling for swallowed InterruptedException thrown by Thread.sleep in 
> MiniZooKeeperCluster
> ----------------------------------------------------------------------------------------------
>
>                 Key: HBASE-10520
>                 URL: https://issues.apache.org/jira/browse/HBASE-10520
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Zookeeper
>            Reporter: Feng Honghua
>            Assignee: Feng Honghua
>            Priority: Minor
>         Attachments: HBASE-10520-trunk_v1.patch, HBASE-10520-trunk_v2.patch, 
> HBASE-10520-trunk_v3.patch
>
>
> A sub-task of HBASE-10497



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to