> On May 15, 2014, 3:01 p.m., Jun Rao wrote: > > core/src/main/scala/kafka/consumer/ConsoleConsumer.scala, lines 165-166 > > <https://reviews.apache.org/r/21239/diff/3/?file=581744#file581744line165> > > > > Since this is a command line tool, we should print the message to > > stdout, instead of logging, so that users can see the message on the > > console.
stdout->stderr - Neha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21239/#review43113 ----------------------------------------------------------- On May 14, 2014, 7:08 p.m., Sriharsha Chintalapani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21239/ > ----------------------------------------------------------- > > (Updated May 14, 2014, 7:08 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1431 > https://issues.apache.org/jira/browse/KAFKA-1431 > > > Repository: kafka > > > Description > ------- > > KAFKA-1431. ConsoleConsumer - Option to clean zk consumer path changed > options --delete-consumer-offsets. It checks if existing offsets for givent > topic & groupId exists and throws error if user didn't specifiy > --delete-consumer-offsets > > > KAFKA-1431.ConsoleConsumer - Option to clean zk consumer path. just checking > consumers/groupId/offsets is good enough, removed for checking topicId under > offsets. > > > Diffs > ----- > > core/src/main/scala/kafka/consumer/ConsoleConsumer.scala > 0f62819be0562f62c0f778bd20ead053f01a6f2f > > Diff: https://reviews.apache.org/r/21239/diff/ > > > Testing > ------- > > > Thanks, > > Sriharsha Chintalapani > >
