Re: Review Request 51047: SAMZA-1000 Fix hello-samza documentation to not use latest branch by default

2016-08-17 Thread Jake Maes
> On Aug. 16, 2016, 10:07 p.m., Yi Pan (Data Infrastructure) wrote: > > docs/startup/hello-samza/versioned/index.md, line 53 > > > > > > We will still need to checkout latest hello-samza, if the instruction > > here

Re: Review Request 50583: SAMZA-954 Improve logging for Samza

2016-08-17 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50583/#review146007 --- Ship it! samza-core/src/main/scala/org/apache/samza/util/Loggi

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 50154: SAMZA-976 Samza REST Documentation

2016-08-17 Thread Jake Maes
> On Aug. 16, 2016, 12:50 a.m., Navina Ramesh wrote: > > docs/learn/tutorials/versioned/samza-rest-getting-started.md, line 73 > > > > > > Should change run-samza-rest-service.sh to be executable (chmod 755) > > Jake

Re: Review Request 50154: SAMZA-976 Samza REST Documentation

2016-08-17 Thread Jake Maes
> On Aug. 16, 2016, 12:50 a.m., Navina Ramesh wrote: > > docs/learn/tutorials/versioned/samza-rest-getting-started.md, line 88 > > > > > > I am running in to an issue of not finding config files in the path > > speci

Re: Review Request 50154: SAMZA-976 Samza REST Documentation

2016-08-17 Thread Jake Maes
> On Aug. 16, 2016, 12:50 a.m., Navina Ramesh wrote: > > docs/learn/tutorials/versioned/samza-rest-getting-started.md, line 88 > > > > > > I am running in to an issue of not finding config files in the path > > speci

Re: Review Request 48356: RFC: Samza as a library

2016-08-17 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48356/#review145967 --- A quick question: w/ the recent discussion, are you planning to re