Re: Review Request 52140: added docs for split deployment

2016-09-21 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/#review149931 --- Fix it, then Ship it! Some minor suggestions. docs/learn/docu

Re: Review Request 52133: reopen a Kafka system producer if send called again

2016-09-21 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52133/ --- (Updated Sept. 22, 2016, 12:56 a.m.) Review request for samza. Changes --

Review Request 52140: added docs for split deployment

2016-09-21 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/ --- Review request for samza. Repository: samza Description --- added docs f

Re: Review Request 52133: reopen a Kafka system producer if send called again

2016-09-21 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52133/#review149925 --- Ship it! Ship It! - Xinyu Liu On Sept. 21, 2016, 8:36 p.m.,

Re: Review Request 51934: reopen a Kafka system producer if send called again

2016-09-21 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51934/ --- (Updated Sept. 21, 2016, 8:37 p.m.) Review request for samza. Bugs: SAMZA-101

Review Request 52133: reopen a Kafka system producer if send called again

2016-09-21 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52133/ --- Review request for samza. Repository: samza Description --- reopen a Kaf

Re: Review Request 52066: change warn about loopback address to a debug

2016-09-21 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52066/#review149877 --- Ship it! Ship It! - Navina Ramesh On Sept. 19, 2016, 8:55 p.

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-09-21 Thread Yi Pan (Data Infrastructure)
> On Sept. 20, 2016, 8:35 p.m., Jagadish Venkatraman wrote: > > samza-operator/src/main/java/org/apache/samza/operators/api/Windows.java, > > line 127 > > > > > > Currently we have 2 flavors of the SessionWindow AP

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-09-21 Thread Yi Pan (Data Infrastructure)
> On Sept. 20, 2016, 8:35 p.m., Jagadish Venkatraman wrote: > > samza-operator/src/main/java/org/apache/samza/operators/api/Windows.java, > > line 127 > > > > > > Currently we have 2 flavors of the SessionWindow AP