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

Feng Honghua updated HBASE-10497:
---------------------------------

    Attachment: HBASE-10497-trunk_v1.patch

patch attached

Note: The handling for InterruptedException thrown by sleep within createTable 
in HBaseAdmin.java is rethrow an encapsulated InterruptedIOException, while 
it's ignored for the one thrown within deleteTable. To keep the previous 
semantic, I just log and call Thread.currentThread.interrupt there. But maybe 
rethrowing an encapsulated InterruptedIOException as in createTable is more 
appropriate.

> Add according handling for swallowed InterruptedException thrown by 
> Thread.sleep under HBase-Client/HBase-Server folders systematically
> ---------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-10497
>                 URL: https://issues.apache.org/jira/browse/HBASE-10497
>             Project: HBase
>          Issue Type: Improvement
>          Components: Client, regionserver
>            Reporter: Feng Honghua
>            Assignee: Feng Honghua
>            Priority: Minor
>         Attachments: HBASE-10497-trunk_v1.patch
>
>
> There are many places where InterruptedException thrown by Thread.sleep are 
> swallowed silently (which are neither declared in the caller method's throws 
> clause nor rethrown immediately) under HBase-Client/HBase-Server folders.
> It'd be better to add standard 'log and call currentThread.interrupt' for 
> such cases.



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

Reply via email to