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

2016-08-16 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50154/#review145962 --- Ship it! Ship It! - Navina Ramesh On Aug. 17, 2016, 1:58 a.m

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

2016-08-16 Thread Navina Ramesh
> 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-16 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50154/ --- (Updated Aug. 17, 2016, 1:58 a.m.) Review request for samza, Boris Shkolnik, Ch

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

2016-08-16 Thread Jake Maes
> On Aug. 16, 2016, 12:50 a.m., Navina Ramesh wrote: > > Reminder: This review is for the doc, not the code. So any code feedback will not be reflected in this review. Instead see https://reviews.apache.org/r/50151/ > On Aug. 16, 2016, 12:50 a.m., Navina Ramesh wrote: > > docs/learn/tutorials

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

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

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

2016-08-16 Thread Jake Maes
> On Aug. 16, 2016, 12:55 a.m., Navina Ramesh wrote: > > One last comment: Can you attach the .png file to the JIRA? Otherwise, the > > local deployment has a broken link. done - Jake --- This is an automatically generated e-mail. To r

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

2016-08-16 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/#review145916 --- docs/startup/hello-samza/versioned/index.md (line 52)

Re: Question on changelog partition mapping

2016-08-16 Thread Yi Pan
Hi, Tommy, Yes. Now I understand what you referred to as "non-determinism". The design of the JobCoordinator has the thought that if "no-previous run is found, we are free to start from scratch" in mind. I think the current solution that you can try is to implement a grouper that will guarantee t

Re: Review Request 51022: rewrite the configs before consuming them in CheckpointTool

2016-08-16 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51022/#review145899 --- Ship it! Ship It! - Navina Ramesh On Aug. 12, 2016, 1:16 a.m

Review Request 51142: SAMZA-967: HDFS System Consumer

2016-08-16 Thread Hai Lu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51142/ --- Review request for samza. Bugs: SAMZA-967 https://issues.apache.org/jira/br