Re: Review Request 41874: SAMZA-837: Support Gradle 2.9

2016-01-05 Thread Aleksandar Bircakovic
> On Jan. 5, 2016, 12:21 a.m., Jake Maes wrote: > > Ship It! > > Jake Maes wrote: > Any idea why the checkstyle behavior changed with the newer gradle? Gradle 2.7 or newer is required for Checkstyle versions 6.8 - 6.12.1. In my opinion Gradle will by default try to use new supported versio

Re: Review Request 41663: SAMZA-843 : Slow start of Samza jobs with large number of containers

2016-01-05 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41663/#review112912 --- Ship it! lgtm. Fix the nits and feel free to commit! Thanks! sam

Re: Review Request 41663: SAMZA-843 : Slow start of Samza jobs with large number of containers

2016-01-05 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41663/ --- (Updated Jan. 5, 2016, 9:50 p.m.) Review request for samza, Boris Shkolnik, Yan

samza split deployment

2016-01-05 Thread Boris Shkolnik
Hi all! I've recently opened a design JIRA (SAMZA-849 ) for split framework deployment. Basic idea is to allow samza core code to be deployed separately from the user job's code. This is useful in case where samza is used as a service, and bug fixes

Re: Review Request 41884: SAMZA-802: KafkaSystemAdmin needs to handle empty topic offsets

2016-01-05 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41884/#review112954 --- Ship it! Logic seems correct. - Boris Shkolnik On Jan. 4, 2016,

Re: Review Request 41912: SAMZA-850: Yarn Job Validation Tool

2016-01-05 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41912/#review112970 --- Ship it! Ship It! - Jake Maes On Jan. 5, 2016, 1:21 a.m., Xinyu