Re: Review Request 44920: SAMZA-881

2016-04-04 Thread Jagadish Venkatraman
> On March 31, 2016, 10:43 p.m., Yi Pan (Data Infrastructure) wrote: > > I have some high level questions w/ the current class layout/hierarchy. > > Will sync up w/ Jagadish in person. Thanks for the great feedback :-) I'll update the revised patch soon. Publishing this, so that I don't loose

Re: Review Request 44920: SAMZA-881

2016-03-31 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/#review126385 --- I have some high level questions w/ the current class layout/hiera

Re: Review Request 44920: SAMZA-881

2016-03-31 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/ --- (Updated March 31, 2016, 10:40 p.m.) Review request for samza, Boris Shkolnik,

Re: Review Request 44920: SAMZA-881

2016-03-25 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/#review125436 --- Fix it, then Ship it! Lots of great improvements in the latest

Re: Review Request 44920: SAMZA-881

2016-03-24 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/ --- (Updated March 24, 2016, 9:41 p.m.) Review request for samza, Boris Shkolnik, C