----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25944/#review65997 -----------------------------------------------------------
core/src/main/scala/kafka/client/ClientUtils.scala <https://reviews.apache.org/r/25944/#comment109304> See comment in OffsetClient core/src/main/scala/kafka/tools/ExportOffsets.scala <https://reviews.apache.org/r/25944/#comment109308> I thought we would be consolidating the export/import tool into a single consumer offsets tool? core/src/main/scala/kafka/tools/OffsetClient.scala <https://reviews.apache.org/r/25944/#comment109306> Ideally, we should have a _class_ OffsetClient - actually, a trait and the interface should be: commitOffsets fetchOffsets close It would be convenient for the user to just instantiate it (the constructor would take a group), use it and then close it. core/src/main/scala/kafka/tools/OffsetClient.scala <https://reviews.apache.org/r/25944/#comment109303> This really should not be part of the offset client's public API. fetchTopicMetadata is a utils-level concept. It can take a channel as an argument over which the metadata request will be sent. - Joel Koshy On Dec. 24, 2014, 12:41 a.m., Mayuresh Gharat wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25944/ > ----------------------------------------------------------- > > (Updated Dec. 24, 2014, 12:41 a.m.) > > > Review request for kafka and joel koshy. > > > Bugs: KAFKA-1013 > https://issues.apache.org/jira/browse/KAFKA-1013 > > > Repository: kafka > > > Description > ------- > > Reverted Changes to ZookeeperConsumerConnector.scala > > > Diffs > ----- > > core/src/main/scala/kafka/client/ClientUtils.scala > ebba87f0566684c796c26cb76c64b4640a5ccfde > core/src/main/scala/kafka/consumer/SimpleConsumer.scala > e53ee51638b930d8592ce9994ae2a617e7ef771c > core/src/main/scala/kafka/javaapi/consumer/SimpleConsumer.scala > 0ab0195dc9f66f407061d0fac2549bd6291e41ff > core/src/main/scala/kafka/tools/ConsumerOffsetChecker.scala > d1e7c434e77859d746b8dc68dd5d5a3740425e79 > core/src/main/scala/kafka/tools/ExportOffsets.scala PRE-CREATION > core/src/main/scala/kafka/tools/ExportZkOffsets.scala > 4d051bc2db12f0e15aa6a3289abeb9dd25d373d2 > core/src/main/scala/kafka/tools/ImportOffsets.scala PRE-CREATION > core/src/main/scala/kafka/tools/ImportZkOffsets.scala > abe09721b13f71320510fd1a01c1917470450c6e > core/src/main/scala/kafka/tools/OffsetClient.scala PRE-CREATION > core/src/main/scala/kafka/tools/OffsetClientConfig.scala PRE-CREATION > core/src/main/scala/kafka/utils/Utils.scala > 738c1af9ef5de16fdf5130daab69757a14c48b5c > core/src/test/scala/other/kafka/TestOffsetManager.scala > 41f334d48897b3027ed54c58bbf4811487d3b191 > core/src/test/scala/unit/kafka/server/OffsetCommitTest.scala > 8c5364fa97da1be09973c176d1baeb339455d319 > > Diff: https://reviews.apache.org/r/25944/diff/ > > > Testing > ------- > > > Thanks, > > Mayuresh Gharat > >