Re: Review Request 52140: added docs for split deployment

2016-09-26 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/#review150500 --- Ship it! Ship It! - Xinyu Liu On Sept. 27, 2016, 12:58 a.m.,

Re: Review Request 52140: added docs for split deployment

2016-09-26 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/ --- (Updated Sept. 27, 2016, 12:58 a.m.) Review request for samza. Changes --

Re: Review Request 52140: added docs for split deployment

2016-09-23 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/ --- (Updated Sept. 23, 2016, 5:24 p.m.) Review request for samza. Bugs: SAMZA-927

Re: Review Request 52140: added docs for split deployment

2016-09-22 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/#review150104 --- docs/learn/documentation/versioned/operations/split-deployment.md

Re: Review Request 52140: added docs for split deployment

2016-09-22 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/ --- (Updated Sept. 22, 2016, 11:43 p.m.) Review request for samza. Bugs: SAMZA-92

Re: Review Request 52140: added docs for split deployment

2016-09-22 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/#review150101 --- Ship it! Ship It! - Xinyu Liu On Sept. 22, 2016, 11:38 p.m.,

Re: Review Request 52140: added docs for split deployment

2016-09-22 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/ --- (Updated Sept. 22, 2016, 11:38 p.m.) Review request for samza. Repository: sa

Re: Review Request 52140: added docs for split deployment

2016-09-22 Thread Boris Shkolnik
> On Sept. 22, 2016, 1:07 a.m., Xinyu Liu wrote: > > docs/learn/documentation/versioned/operations/split-deployment.md, line 48 > > > > > > seems here are two "split deployment" bullets here. Do we want to > > distin

Re: Review Request 52140: added docs for split deployment

2016-09-21 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52140/#review149931 --- Fix it, then Ship it! Some minor suggestions. docs/learn/docu