Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/#review147542 --- ambari-metrics/ambari-metrics-host-monitoring/src/main/python/cor

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/#review147536 --- Ship it! Ship It! - Sid Wagle On Sept. 1, 2016, 3:51 a.m., A

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/ --- (Updated Sept. 1, 2016, 3:51 a.m.) Review request for Ambari, Dmytro Sen, Sumit

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/#review147528 --- ambari-metrics/ambari-metrics-host-monitoring/src/main/python/cor

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/ --- (Updated Aug. 31, 2016, 11:41 p.m.) Review request for Ambari, Dmytro Sen, Sumi

Re: Review Request 51370: DB consistency check should tolerate services with no configuration

2016-08-31 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51370/#review147484 --- What's the state of this review? If it's pushed can it be closed?

Re: Review Request 51538: AMBARI-18280 Ambari widget graph is not aggregating the data correctly for Total Request Count

2016-08-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51538/#review147478 --- Ship it! Ship It! - Aravindan Vijayan On Aug. 30, 2016, 7:46

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/#review147476 --- Sumit, i didn't add code which will remove hcat_user property duri

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/ --- (Updated Сер. 31, 2016, 7:33 після полудня) Review request for Ambari, Andrew O

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/#review147474 --- Sumit, i didn't add code which will remove hcat_user property duri

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/#review147467 --- ambari-metrics/ambari-metrics-host-monitoring/src/main/python/cor

Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-08-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Bugs: AMBA

Re: Review Request 51540: Expose Disabling of Alert Targets in Web Client

2016-08-31 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51540/#review147460 --- Ship it! Ship It! - Jaimin Jetly On Aug. 31, 2016, 1:51 a.m.

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/ --- (Updated Сер. 31, 2016, 6:29 після полудня) Review request for Ambari, Andrew O

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/#review147446 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 30, 2016, 4:0

Re: Review Request 51524: Hive service check failed

2016-08-31 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51524/#review147436 --- Ship it! Ship It! - Dmytro Sen On Авг. 30, 2016, 1:02 п.п.,

Re: Review Request 51550: [AMBARI-18284] Ambari PigView stores the pig scripts in HDFS in non UTF-8 format

2016-08-31 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51550/#review147432 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Aug. 31, 2016, 8:08

Re: Review Request 51550: [AMBARI-18284] Ambari PigView stores the pig scripts in HDFS in non UTF-8 format

2016-08-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51550/#review147427 --- Ship it! Ship It! - Jonathan Hurley On Aug. 31, 2016, 4:08 a

Re: Review Request 51522: Add logging to differentiate between ambari-server start/restart commands vs. other commands

2016-08-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51522/#review147426 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 30, 2016, 8:1

Re: Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/#review147425 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 31, 2016, 2:3

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-08-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/#review147424 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 31, 2016, 3:2

Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, and

Re: Review Request 51550: [AMBARI-18284] Ambari PigView stores the pig scripts in HDFS in non UTF-8 format

2016-08-31 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51550/#review147423 --- Ship it! Ship It! - Nate Cole On Aug. 31, 2016, 4:08 a.m., J

Re: Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/#review147422 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 31, 2016, 11:

Re: Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/ --- (Updated Aug. 31, 2016, 11:33 a.m.) Review request for Ambari and Dmitro Lisnic

Re: Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/ --- (Updated Aug. 31, 2016, 11:20 a.m.) Review request for Ambari and Dmitro Lisnic

Re: Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/#review147421 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 31, 2016, 11:

Review Request 51556: Processes children are not killed on timeout

2016-08-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51556/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-18286 https:

Re: Review Request 51555: Ambari upgrade from 2.4.0.x version fails.

2016-08-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51555/#review147420 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 31, 2016, 11:

Review Request 51555: Ambari upgrade from 2.4.0.x version fails.

2016-08-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51555/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-18285 https:

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-08-31 Thread Akhil PB
> On Aug. 31, 2016, 8:43 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/router.js, line > > 270 > > > > > > - Why are we having this controller variable > > -

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-08-31 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51452/#review147417 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/router