[ https://issues.apache.org/jira/browse/CASSANDRA-5623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13681239#comment-13681239
]
Dave Brosius commented on CASSANDRA-5623:
-----------------------------------------
thanks for the clarification.
my concern is that how do i, in a code review, determine if this doesn't introduce infinite
loops, that were once broken out of by the IE. Also, fail fast tends to have safer/easier
to recover from behavior.
This patch might be fine, I'm just not sure how to prove it to myself.
> Cleanup interruption handling
> -----------------------------
>
> Key: CASSANDRA-5623
> URL: https://issues.apache.org/jira/browse/CASSANDRA-5623
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Mikhail Mazursky
> Assignee: Mikhail Mazursky
> Priority: Minor
> Fix For: 2.0
>
> Attachments: trunk-5623.txt
>
>
> There are a lot of catch-wrap-throw pattern occurances in code with InterruptedException.
I cleaned up some of them but not all - in some places I don't know enough about code to decide
if it's correct thing to do.
> Important: I also fixed possibility of wrong behaviour in case of spurious wakeup in
AsyncOneResponse.
--
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
|