You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2013/05/11 23:01:16 UTC

[jira] [Commented] (CASSANDRA-5557) Correctly handle interruptions of Thread.sleep()

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

Jonathan Ellis commented on CASSANDRA-5557:
-------------------------------------------

Fine w/ using sleepUninterruptibly.  Not fine w/ up-propagating Exception.
                
> Correctly handle interruptions of Thread.sleep()
> ------------------------------------------------
>
>                 Key: CASSANDRA-5557
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5557
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Mikhail Mazursky
>            Priority: Minor
>         Attachments: trunk-5557.txt
>
>
> There is no need to throw assertion errors or runtime exceptions when Thread.sleep() is interrupted - code can just ignore interruption (and that is correct behaviour). Also this eliminates boilerplate code. See attached patch.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira