hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-10455) cleanup InterruptedException management
Date Mon, 03 Feb 2014 19:29:05 GMT

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

Andrew Purtell commented on HBASE-10455:
----------------------------------------

TestZKSecretWatcher may need fixing after this, if the precommit failure is not spurious.
This can be done as a follow up. Open an issue and assign it to me or I'll take care of it
if I see it.

> cleanup InterruptedException management
> ---------------------------------------
>
>                 Key: HBASE-10455
>                 URL: https://issues.apache.org/jira/browse/HBASE-10455
>             Project: HBase
>          Issue Type: Bug
>          Components: Client, master, regionserver
>    Affects Versions: 0.98.0, 0.99.0
>            Reporter: Nicolas Liochon
>            Assignee: Nicolas Liochon
>             Fix For: 0.98.0, 0.99.0
>
>         Attachments: 10455.v1.patch
>
>
> 4 changes in this code:
> 1) When caught and rethrowed as a IOException we always rethrow InterruptedIOException
> 2) When we were both throwing an exception AND resetting the interrupt status we only
throw an exception now.
> 3) When we were trying to reset the status by Thread.interrupted() (which does not do
that), we now do it for real with a Thread.currentThread.interrupt().
> 4) Sometimes, we were rethrowing something else then InterruptedIOException, while the
contract would have allowed it. I've changed this as well.
> This patch does not make means that we're fine when we're interrupted, but we're globally
cleaner at least. I will then create other patches specific to some parts.



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

Mime
View raw message