Re: Review Request 49484: Remove spark.yarn.max.executor.failures configuration in Spark Ambari definition (2.4/2.5 branch)

2016-10-19 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49484/#review153348 --- Ship it! Ship It! - Sumit Mohanty On Oct. 20, 2016, 3:12

Re: Review Request 49484: Remove spark.yarn.max.executor.failures configuration in Spark Ambari definition (2.4/2.5 branch)

2016-10-19 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49484/ --- (Updated Oct. 20, 2016, 3:12 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 49484: Remove spark.yarn.max.executor.failures configuration in Spark Ambari definition

2016-10-19 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49484/ --- (Updated Oct. 20, 2016, 3:07 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 52842: AMBARI-18593 : Provide ability to use downsampling function on certain metrics like client side topN

2016-10-19 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52842/#review153330 ---

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-19 Thread Amarnath reddy pappu
> On Oct. 18, 2016, 9:15 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 2428 > > > > > > nit: "Indicates the delay, in milliseconds, for

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-19 Thread Amarnath reddy pappu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/ --- (Updated Oct. 19, 2016, 10:51 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53030: Ambari upgrade failed while running 'Alter Table blueprint' - blueprint_name column

2016-10-19 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53030/#review153306 --- Ship it! Ship It! - Sumit Mohanty On Oct. 19, 2016, 7:17

Review Request 53030: Ambari upgrade failed while running 'Alter Table blueprint' - blueprint_name column

2016-10-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53030/ --- Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and Sid Wagle.

Re: Review Request 52964: Optimize Service Checks to it picks a random host and prefers hosts with 0 active commands

2016-10-19 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52964/ --- (Updated Oct. 19, 2016, 5:52 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 52964: Optimize Service Checks to it picks a random host and prefers hosts with 0 active commands

2016-10-19 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52964/ --- (Updated Oct. 19, 2016, 5:48 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 52964: Optimize Service Checks to it picks a random host and prefers hosts with 0 active commands

2016-10-19 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52964/#review153297 --- Ship it! Ship It! - Sid Wagle On Oct. 19, 2016, 1:35 a.m.,

Re: Review Request 52996: For rolling upgrade of Kafka 0.10.0.1, set configs for backward compatibility

2016-10-19 Thread Nate Cole
> On Oct. 18, 2016, 10:52 p.m., Jonathan Hurley wrote: > > Out of curiousity, instead of providing a message about removing the two > > properties, could we just have a new stage right before finalization that > > does it automatically? I know it won't restart anything, but at least the > >

Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-19 Thread Akhil PB
> On Oct. 12, 2016, 8:35 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/editqueue.js, > > line 120 > > > > > > Why is this conditional operator

Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-19 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52688/ --- (Updated Oct. 19, 2016, 11:42 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-19 Thread Attila Doroszlai
> On Oct. 18, 2016, 7:37 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > lines 953-954 > > > > > > Variable names `monitor_delay` and