Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-08-22 Thread Jake Maes
> On Aug. 23, 2016, 1:21 a.m., Navina Ramesh wrote: > > samza-rest/src/main/java/org/apache/samza/monitor/MonitorLoader.java, line > > 23 > > > > > > I wonder if it will be useful to have this as a more generic Utili

Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-08-22 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50151/#review145963 --- Ship it! Overall looks good. Just jotted down some thoughts :)

Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-08-17 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50151/ --- (Updated Aug. 17, 2016, 3:35 p.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-07-18 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50151/ --- (Updated July 18, 2016, 7:30 p.m.) Review request for samza, Boris Shkolnik, Ch

Review Request 50151: SAMZA-975 Initial Samza REST Implementation

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