You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Jason Gustafson (JIRA)" <ji...@apache.org> on 2015/12/01 02:23:11 UTC

[jira] [Commented] (KAFKA-2911) Replace Utils.sleep() with Time.sleep() whenever possible

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

Jason Gustafson commented on KAFKA-2911:
----------------------------------------

There were only a few places in the new consumer, so I went ahead and fixed this in the patch for KAFKA-2880.

> Replace Utils.sleep() with Time.sleep() whenever possible
> ---------------------------------------------------------
>
>                 Key: KAFKA-2911
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2911
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Guozhang Wang
>              Labels: newbie
>             Fix For: 0.9.1.0
>
>
> In our Java clients we have a mixed usage of {code}Utils.sleep(){code} and {code}Time.sleep(){code}. We should use the latter case where possible, since it allows us to take advantage of MockTime in test cases.
> We can make a pass over the code to see whether any uses of Utils.sleep can be replaced.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)