Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/ --- (Updated May 12, 2016, 12:32 a.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/ --- (Updated May 11, 2016, 11:55 p.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/ --- (Updated May 11, 2016, 11:49 p.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/#review132785 --- Ship it! samza-core/src/main/java/org/apache/samza/coordinator

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/#review132742 --- Ship it! lgtm. samza-core/src/main/java/org/apache/samza/coor

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Jake Maes
> On May 11, 2016, 6:28 p.m., Chris Pettitt wrote: > > Yes, and yes. Thanks! - Jake --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/#review132710 ---

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/ --- (Updated May 11, 2016, 8:13 p.m.) Review request for samza, Boris Shkolnik, Chr

Re: Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-11 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/#review132710 --- Fix it, then Ship it! samza-core/src/main/java/org/apache/samz

Review Request 47197: SAMZA-948 CoordinatorSystemStreamConsumer is not threadsafe

2016-05-10 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47197/ --- Review request for samza, Boris Shkolnik, Chris Pettitt, Jake Maes, Navina Rames