> On March 6, 2014, 6:47 p.m., Jun Rao wrote: > > core/src/main/scala/kafka/producer/ConsoleProducer.scala, lines 115-126 > > <https://reviews.apache.org/r/18712/diff/4/?file=511668#file511668line115> > > > > Perhaps we just need to keep one of the two options and set different > > properties in the old and the new producer config.
Good point, will fix during checkin. - Neha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18712/#review36390 ----------------------------------------------------------- On March 6, 2014, 12:32 a.m., Neha Narkhede wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18712/ > ----------------------------------------------------------- > > (Updated March 6, 2014, 12:32 a.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1281 > https://issues.apache.org/jira/browse/KAFKA-1281 > > > Repository: kafka > > > Description > ------- > > Reverting change to log4j.properties > > > Addressed Jun's review comments. At a high level, this diff changes all tools > to use response.get() when the --sync option is used > > > Found a few problems with the current tools. 1) Moved TestLogCleaning and > TestEndToEndLatency to under tools so we can run those using > bin/kafka-run-class.sh 2) TestLogCleaning seems to hang while sending data > using the new producer. Will file a JIRA to track this > > > First pass at refactoring tools to use the new producer > > > Diffs > ----- > > clients/src/main/java/org/apache/kafka/clients/producer/ProducerConfig.java > 307659cf1abacf7933c02e38d0b5a00233bad355 > config/tools-log4j.properties 79240490149835656e2a013a9702c5aa41c104f1 > core/src/main/scala/kafka/producer/ConsoleProducer.scala > 28de57304db6a7c79c4c75de155f6389144eb5a5 > core/src/main/scala/kafka/producer/KafkaLog4jAppender.scala > 42239b27ab26725141f4bf525d2d69538979dd44 > core/src/main/scala/kafka/tools/ReplayLogProducer.scala > 814d61ae477cef5e56723e1f1a86c3b2e9b7c1ea > core/src/main/scala/kafka/utils/CommandLineUtils.scala > 726c30257dcb55b8989163dab3bfc6888dd5d931 > core/src/test/scala/other/kafka/TestEndToEndLatency.scala > f5d39dd71ad25b0d9955d4b9bcdb56ba705abff0 > core/src/test/scala/other/kafka/TestLogCleaning.scala > d20d132de6ef1101a0d8cade9a5417e4286e6117 > > Diff: https://reviews.apache.org/r/18712/diff/ > > > Testing > ------- > > > Thanks, > > Neha Narkhede > >
