Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Hai Lu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52570/ --- (Updated Jan. 28, 2017, 6:15 a.m.) Review request for samza. Bugs: SAMZA-1025

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52570/#review163355 --- Fix it, then Ship it! Some nits and comments. Otherwise, looks

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Navina Ramesh
> On Jan. 25, 2017, 10:36 p.m., Jagadish Venkatraman wrote: > > docs/learn/documentation/versioned/hdfs/consumer.md, line 67 > > > > > > The relationship between whitelist and blacklist was not very obvious > > to me

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Navina Ramesh
> On Jan. 25, 2017, 10:50 p.m., Navina Ramesh wrote: > > docs/learn/documentation/versioned/hdfs/consumer.md, line 26 > > > > > > Can you include the diagram from your design document? Or something > > similar to el

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52570/#review16 --- Ship it! Thanks! docs/learn/documentation/versioned/hdfs/cons

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Hai Lu
> On Jan. 27, 2017, 7:03 a.m., Jagadish Venkatraman wrote: > > This is looking pretty good. Thank you for the effort in writing the docs! Thank you for taking the time to review it. Really appreciate it:) > On Jan. 27, 2017, 7:03 a.m., Jagadish Venkatraman wrote: > > docs/learn/documentation/v

Re: Review Request 52570: SAMZA-1025: documentation for hdfs system consumer

2017-01-27 Thread Hai Lu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52570/ --- (Updated Jan. 27, 2017, 5:48 p.m.) Review request for samza. Bugs: SAMZA-1025