----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24510/#review50127 -----------------------------------------------------------
Thanks for the patch. Could you take a look at the following comment? system_test/utils/kafka_system_test_utils.py <https://reviews.apache.org/r/24510/#comment87703> Does this work as expected? I tried the following test. However, the pid in the file tt is not the pid of the sleep process. echo $! > tt & sleep 100 - Jun Rao On Aug. 8, 2014, 9:50 p.m., Dong Lin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24510/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2014, 9:50 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1582 > https://issues.apache.org/jira/browse/KAFKA-1582 > > > Repository: kafka > > > Description > ------- > > KAFKA-1582; System Test should wait for producer to finish > > > Diffs > ----- > > system_test/utils/kafka_system_test_utils.py > 6edd64a92ce749421b5ab60fbea5b55e2ccbfe94 > > Diff: https://reviews.apache.org/r/24510/diff/ > > > Testing > ------- > > > Thanks, > > Dong Lin > >
