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)