[jira] [Commented] (KAFKA-2247) Merge kafka.utils.Time and kafka.common.utils.Time

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on KAFKA-2247:
---------------------------------------

GitHub user ijuma opened a pull request:

    https://github.com/apache/kafka/pull/2095

    KAFKA-2247: Merge kafka.utils.Time and kafka.common.utils.Time

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ijuma/kafka kafka-2247-consolidate-time-interfaces

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/2095.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2095
    
----
commit 5126636282ca234ba9945acf9959d3182b20b33c
Author: Ismael Juma <is...@juma.me.uk>
Date:   2016-11-03T10:17:54Z

    KAFKA-2247: Merge kafka.utils.Time and kafka.common.utils.Time

----


> Merge kafka.utils.Time and kafka.common.utils.Time
> --------------------------------------------------
>
>                 Key: KAFKA-2247
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2247
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Aditya Auradkar
>            Assignee: Ismael Juma
>            Priority: Minor
>
> We currently have 2 different versions of Time in clients and core. These need to be merged.
> It's worth noting that `kafka.utils.MockTime` includes a `scheduler` that is used by some tests while `o.a.kafka.common.utils.Time` does not. We either need to add this functionality or change the tests not to need it anymore.



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