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

2016-08-18 Thread Navina Ramesh
> 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 48356: RFC: Samza as a library

2016-08-18 Thread Navina Ramesh
> On Aug. 17, 2016, 8:36 a.m., Yi Pan (Data Infrastructure) wrote: > > A quick question: w/ the recent discussion, are you planning to remove the > > ConfigBuilder interface and add a StreamProcessorFactory? Yes Yi. As we discussed online, I think we should commit this RB (as is) in order to u

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

2016-08-18 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48356/ --- (Updated Aug. 18, 2016, 5:39 p.m.) Review request for samza, Chris Pettitt and

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

2016-08-18 Thread Navina Ramesh
> On July 15, 2016, 12:12 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-core/src/main/java/org/apache/samza/configbuilder/ConfigBuilder.java, > > line 31 > > > > > > Just wonder, can we have an interface class

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

2016-08-18 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 51142: SAMZA-967: HDFS System Consumer

2016-08-18 Thread Hai Lu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51142/ --- (Updated Aug. 18, 2016, 6:05 p.m.) Review request for samza. Bugs: SAMZA-967

[DISCUSS] A HDFS system consumer for Samza

2016-08-18 Thread Hai Lu
Hi, I have been recently working on a HDFS system consumer for Samza. The work includes two major parts: 1. properly partitioning a HDFS directory and 2. consuming from HDFS files. I have attached the design doc in the Jira ticket here: https://issues.apache.org/jira/browse/SAMZA-967 It would be

Review Request 51225: SAMZA-1003: Restore lazy init for kafka system producer

2016-08-18 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51225/ --- Review request for samza, Jake Maes and Yi Pan (Data Infrastructure). Repositor

Re: Review Request 51225: SAMZA-1003: Restore lazy init for kafka system producer

2016-08-18 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51225/#review146138 --- Ship it! Ship It! - Jake Maes On Aug. 18, 2016, 10:12 p.m.,

Re: Review Request 51225: SAMZA-1003: Restore lazy init for kafka system producer

2016-08-18 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51225/#review146145 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On Aug. 18,

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

2016-08-18 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/ --- (Updated Aug. 19, 2016, 12:04 a.m.) Review request for samza, Boris Shkolnik, C

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

2016-08-18 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 51047: SAMZA-1000 Fix hello-samza documentation to not use latest branch by default

2016-08-18 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/#review146152 --- gradle.properties (line 18)

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

2016-08-18 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/ --- (Updated Aug. 19, 2016, 12:27 a.m.) Review request for samza, Boris Shkolnik, C

Review Request 51228: SAMZA-1000 Update readme for hello-samza with a local build

2016-08-18 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51228/ --- Review request for samza, Boris Shkolnik, Chris Pettitt, Fred Ji, Jake Maes, Nav