Re: Review Request 48121: YARN default configs are invalid

2016-06-03 Thread Dmytro Sen
> On Июнь 2, 2016, 4:55 п.п., Robert Nettleton wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 337 > > > > > > This code looks fine

Re: Review Request 48121: YARN default configs are invalid

2016-06-03 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/ --- (Updated Июнь 3, 2016, 10:43 д.п.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48121: YARN default configs are invalid

2016-06-02 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/#review135962 --- Fix it, then Ship it! The patch looks fine to me, just a minor

Re: Review Request 48121: YARN default configs are invalid

2016-06-02 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/#review135959 --- Ship it! Ship It! - Sebastian Toader On June 2, 2016, 6:38

Re: Review Request 48121: YARN default configs are invalid

2016-06-01 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/#review135809 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 1, 2016,

Re: Review Request 48121: YARN default configs are invalid

2016-06-01 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/ --- (Updated Июнь 1, 2016, 1:16 п.п.) Review request for Ambari, Andrew Onischuk