[jira] [Comment Edited] (CASSANDRA-7149) SimpleCondition await bug

["Jianwei Zhang (JIRA)" <ji...@apache.org>]

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

Jianwei Zhang edited comment on CASSANDRA-7149 at 5/4/14 11:04 AM:
-------------------------------------------------------------------

patch for 1.2 is attatched

this bug has no bad influence in the 1.x versions, because all the reference to await use TimeUnit.MILLISECONDS as the unit,  like this:
condition.await(timeout, TimeUnit.MILLISECONDS);


was (Author: jianwei zhang):
patch for 1.2 is attatched

> SimpleCondition await bug
> -------------------------
>
>                 Key: CASSANDRA-7149
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7149
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Jianwei Zhang
>            Priority: Minor
>             Fix For: 1.2.17
>
>         Attachments: cassandra-1.2-7149.txt
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> SimpleCondition await
> There is a bug when converting other time unit to millisecond .
> now:
> long end = System.currentTimeMillis() + unit.convert(time, TimeUnit.MILLISECONDS);
> should be:
> long end = System.currentTimeMillis() + TimeUnit.MILLISECONDS.convert(time, unit);



--
This message was sent by Atlassian JIRA
(v6.2#6252)