----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19510/#review39068 -----------------------------------------------------------
Not sure if I'm missing something here, but I don't see any Popen.communicate changes in this rb? - Timothy Chen On March 31, 2014, 5:32 p.m., Jun Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19510/ > ----------------------------------------------------------- > > (Updated March 31, 2014, 5:32 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1318 and KAFKA-1318-v2 > https://issues.apache.org/jira/browse/KAFKA-1318 > https://issues.apache.org/jira/browse/KAFKA-1318-v2 > > > Repository: kafka > > > Description > ------- > > 1. Used Popen.communicate to wait for producer to stop. 2. Added the > log4j.properties for migration tool testsuite. 3. Passed in the right log4j > property file for the broker. 4. Removed topic creating in migration tool > test script since it's trying to run the 0.8 topic creation script on the 0.7 > broker. Instead, increased retry.backoff.ms in migration producer to avoid > data loss on auto created topic. 5. Improved README. > > > Diffs > ----- > > system_test/README.txt 87937ecd7d637c62f2d401f54f1d669ac0d13f0a > system_test/migration_tool_testsuite/0.7/config/log4j.properties > PRE-CREATION > system_test/migration_tool_testsuite/config/migration_producer.properties > 17b5928a1213fc0cae7cd94cd214323ccca61d48 > system_test/migration_tool_testsuite/migration_tool_test.py > 2386a5884fcf86bac282ef28f07a3af740185f7f > system_test/utils/kafka_system_test_utils.py > 35f2d1b0464edb738cd8e01f8654913776c49459 > > Diff: https://reviews.apache.org/r/19510/diff/ > > > Testing > ------- > > > Thanks, > > Jun Rao > >