Re: Review Request 32155: SAMZA-458: Close in KafkaSystemProducer should flush all source buffers

2015-03-20 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32155/#review77293 --- Ship it! Ship It! - Navina Ramesh On March 20, 2015, 9:40 p.m.,

Re: Review Request 32155: SAMZA-458: Close in KafkaSystemProducer should flush all source buffers

2015-03-20 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32155/#review77282 --- Ship it! Looks good to me. - Yi Pan (Data Infrastructure) On Mar

Review Request 32332: SAMZA-367

2015-03-20 Thread Yan Fang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32332/ --- Review request for samza. Bugs: SAMZA-367 https://issues.apache.org/jira/br

Re: Review Request 32155: SAMZA-458: Close in KafkaSystemProducer should flush all source buffers

2015-03-20 Thread Yan Fang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32155/ --- (Updated March 20, 2015, 9:40 p.m.) Review request for samza. Bugs: SAMZA-458

Review Request 32261: SAMZA-588 - Expose SamzaContainerContext through TaskContext

2015-03-20 Thread Benjamin Fradet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32261/ --- Review request for samza. Repository: samza Description --- SAMZA-588 -

Re: Review Request 32155: SAMZA-458: Close in KafkaSystemProducer should flush all source buffers

2015-03-20 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32155/#review77245 --- samza-kafka/src/main/scala/org/apache/samza/system/kafka/KafkaSyste