[ 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 <ism...@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)