Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Kafka >> mail # dev >> Review Request 21616: Patch for KAFKA-924


Copy link to this message
-
Re: Review Request 21616: Patch for KAFKA-924

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21616/#review44232
There seems to be a bunch of white space changes. Could we remove them?
system_test/utils/kafka_system_test_utils.py
<https://reviews.apache.org/r/21616/#comment78568>

    The comment probably needs to be changed.

system_test/utils/kafka_system_test_utils.py
<https://reviews.apache.org/r/21616/#comment78566>

    Do we need space btw props_file_path and hostname?

system_test/utils/kafka_system_test_utils.py
<https://reviews.apache.org/r/21616/#comment78567>

    Do we still need groupOption here?
- Jun Rao
On May 29, 2014, 12:47 a.m., Sriharsha Chintalapani wrote: