Re: Review Request 46148: NPE when deleting a host

2016-04-14 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/#review128869 --- Ship it! Ship It! - Sebastian Toader On April 14, 2016,

Review Request 46195: ATS heapsize cannot be changed in Ambari-2.2.1.1 or greater post Ambari Server upgrade, even though the bug should be fixed

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

Re: Review Request 46148: NPE when deleting a host

2016-04-14 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/ --- (Updated ápr. 14, 2016, 10:25 de) Review request for Ambari, Laszlo Puskas,

Review Request 46194: HDP 2.3.2.0: Changing hive tez container size (via ambari) should change tez.runtime.io.sort.mb as well

2016-04-14 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46194/ --- Review request for Ambari and Srimanth Gunturi. Bugs: AMBARI-15879

Re: Review Request 46195: ATS heapsize cannot be changed in Ambari-2.2.1.1 or greater post Ambari Server upgrade, even though the bug should be fixed

2016-04-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46195/#review128878 --- Ship it! Ship It! - Vitalyi Brodetskyi On April 14, 2016,

Re: Review Request 46148: NPE when deleting a host

2016-04-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/#review128867 --- Ship it! Ship It! - Oliver Szabo On April 14, 2016, 10:25

Re: Review Request 46148: NPE when deleting a host

2016-04-14 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/ --- (Updated ápr. 14, 2016, 11:41 de) Review request for Ambari, Laszlo Puskas,

Re: Review Request 46148: NPE when deleting a host

2016-04-14 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/#review128880 --- Ship it! Ship It! - Sebastian Toader On April 14, 2016,

Re: Review Request 46218: AlertReceivedListenerTest is Flaky When Running @Before Code

2016-04-14 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46218/#review128964 --- Ship it! Ship It! - Nate Cole On April 14, 2016, 3:27 p.m.,

Re: Review Request 46169: Add dryrun method on request object

2016-04-14 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46169/#review128965 --- Ship it! Ship It! - Nate Cole On April 14, 2016, 10:30

Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46219/ --- Review request for Ambari, Andrii Tkach, Dmytro Sen, Jaimin Jetly, Sumit

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
> On April 14, 2016, 3:33 a.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/common.py, > > line 92 > > > > > > I think this function should only take a

Review Request 46217: After adding support for real jdbc jar names, verify the post Ambari upgrade nothing breaks

2016-04-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46217/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.

Review Request 46218: AlertReceivedListenerTest is Flaky When Running @Before Code

2016-04-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46218/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Sid Wagle.

Review Request 46221: RU/EU: Unable to start upgrade with host in MM

2016-04-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46221/ --- Review request for Ambari, Alejandro Fernandez and Nate Cole. Bugs:

Re: Review Request 46218: AlertReceivedListenerTest is Flaky When Running @Before Code

2016-04-14 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46218/#review128968 --- Ship it! Ship It! - Sid Wagle On April 14, 2016, 7:27 p.m.,

Re: Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46219/#review128967 --- Ship it! Ship It! - Sid Wagle On April 14, 2016, 7:40 p.m.,

Re: Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46219/#review128977 ---

Re: Review Request 46221: RU/EU: Unable to start upgrade with host in MM

2016-04-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46221/#review128986 --- Ship it! Ship It! - Alejandro Fernandez On April 14, 2016,

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/ --- (Updated April 14, 2016, 9:20 p.m.) Review request for Ambari. Changes

Re: Review Request 46226: AMBARI-15899 Alert Check Retries: UX edits v1

2016-04-14 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46226/#review129014 ---

Re: Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46219/ --- (Updated April 14, 2016, 8:56 p.m.) Review request for Ambari, Andrii Tkach,

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/#review129024 ---

Re: Review Request 46170: HAWQ Segment Registration Status Alert shows false alert

2016-04-14 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46170/#review129033 --- Ship it! Ship It! - Goutam Tadi On April 14, 2016, 11:52

Re: Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Aravindan Vijayan
> On April 14, 2016, 8:28 p.m., Jaimin Jetly wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/widgets.json, > > lines 201-205 > > > > > > Lets remove this mem_cached entry if it is not

Review Request 46225: Change HDP-2.5 Hive Theme to be extended from the base theme

2016-04-14 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46225/ --- Review request for Ambari, Richard Zang, Srimanth Gunturi, and Yusaku Sako.

Re: Review Request 46219: AMBARI-15892 : Incorrect (Negative) values are shown for memory metrics

2016-04-14 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46219/#review128996 --- Ship it! Ship It! - Jaimin Jetly On April 14, 2016, 8:56

Re: Review Request 46170: HAWQ Segment Registration Status Alert shows false alert

2016-04-14 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46170/ --- (Updated April 14, 2016, 11:52 p.m.) Review request for Ambari, Alexander

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/#review128998 --- Ship it! Ship It! - Alejandro Fernandez On April 14, 2016,

Review Request 46226: AMBARI-15899 Alert Check Retries: UX edits v1

2016-04-14 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46226/ --- Review request for Ambari, Alexandr Antonenko, Jaimin Jetly, Jonathan Hurley,

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
> On April 14, 2016, 11 p.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/utils.py, > > line 26 > > > > > > do we need unit test for this function ?

Review Request 46238: Refactor Metadata manager for supporting distributed collector

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

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/ --- (Updated April 15, 2016, midnight) Review request for Ambari. Bugs:

Re: Review Request 46172: Pass request object to delete API in resource providers

2016-04-14 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46172/#review129039 --- Ship it! Ship It! - Jayush Luniya On April 14, 2016, 4:04

Re: Review Request 46169: Add dryrun method on request object

2016-04-14 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46169/#review129037 --- Ship it! Ship It! - Jayush Luniya On April 14, 2016, 2:30

Review Request 46248: YARN work preserving restart should be recommended and validated to be enabled if Hive interactive mode is turned ON

2016-04-14 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46248/ --- Review request for Ambari, Jaimin Jetly and Swapan Shridhar. Bugs:

Re: Review Request 46226: AMBARI-15899 Alert Check Retries: UX edits v1

2016-04-14 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46226/ --- (Updated April 15, 2016, 12:14 a.m.) Review request for Ambari, Alexandr

Re: Review Request 46170: HAWQ Segment Registration Status Alert shows false alert

2016-04-14 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46170/ --- (Updated April 15, 2016, 12:05 a.m.) Review request for Ambari, Alexander

Re: Review Request 46226: AMBARI-15899 Alert Check Retries: UX edits v1

2016-04-14 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46226/#review129046 --- Ship it! Ship It! - Richard Zang On April 15, 2016, 12:14

Review Request 46247: AMBARI-15903 Script + Server alert display, descriptions and ui

2016-04-14 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46247/ --- Review request for Ambari, Alexandr Antonenko, Jaimin Jetly, Jonathan Hurley,

Review Request 46016: AMBARI-15752 Ambari support for additional config params for Ranger KMS to support HSM

2016-04-14 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46016/ --- Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Jonathan Hurley,

Re: Review Request 46183: AMBARI-15876. Stack advisor work for adding and removing LLAP queue when Interactive query is turned on and off respectively.

2016-04-14 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46183/ --- (Updated April 14, 2016, 9:01 a.m.) Review request for Ambari, Alejandro

Re: Review Request 46169: Add dryrun method on request object

2016-04-14 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46169/ --- (Updated April 14, 2016, 2:30 p.m.) Review request for Ambari, Jayush Luniya,

Re: Review Request 46207: Owner of /tmp hdfs directory is yarn user on secure blueprint install

2016-04-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46207/#review128899 --- Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 14, 2016,

Review Request 46207: Owner of /tmp hdfs directory is yarn user on secure blueprint install

2016-04-14 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46207/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-15885

Re: Review Request 46207: Owner of /tmp hdfs directory is yarn user on secure blueprint install

2016-04-14 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46207/ --- (Updated April 14, 2016, 3:04 p.m.) Review request for Ambari and Sumit

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/#review128975 --- Ship it! Ship It! - bhuvnesh chaudhary On April 14, 2016,

Re: Review Request 46016: AMBARI-15752 Ambari support for additional config params for Ranger KMS to support HSM

2016-04-14 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46016/ --- (Updated April 15, 2016, 5:34 a.m.) Review request for Ambari, Alejandro

Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/ --- Review request for Ambari, Robert Levas, Sandor Magyari, and Sebastian Toader.

Re: Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/#review128886 --- Ship it! Ship It! - Robert Nettleton On April 14, 2016,

Re: Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/#review128892 --- Ship it! Ship It! - Sandor Magyari On April 14, 2016, 12:27

Re: Review Request 46200: Support for multiple Namservices in namenode_ha_utils.py

2016-04-14 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46200/#review128891 --- Ship it! Ship It! - Andrew Onischuk On April 14, 2016, 1:13

Re: Review Request 46196: Logsearch: reading config values from external property files

2016-04-14 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46196/#review128894 --- Ship it! Ship It! - Robert Nettleton On April 14, 2016,

Re: Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/#review128893 --- Ship it! Ship It! - Robert Levas On April 14, 2016, 8:27

Review Request 46199: AMBARI-15866: Renew kerberos keyfile rest API returnes a malformed /request URL

2016-04-14 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46199/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-15866

Review Request 46200: Support for multiple Namservices in namenode_ha_utils.py

2016-04-14 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46200/ --- Review request for Ambari, Andrew Onischuk, Sumit Mohanty, and Vitalyi

Review Request 46172: Pass request object to delete API in resource providers

2016-04-14 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46172/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Re: Review Request 46194: HDP 2.3.2.0: Changing hive tez container size (via ambari) should change tez.runtime.io.sort.mb as well

2016-04-14 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46194/#review128918 --- Ship it! Ship It! - Srimanth Gunturi On April 14, 2016,

Re: Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/ --- (Updated April 14, 2016, 3:16 p.m.) Review request for Ambari, Robert Levas,

Re: Review Request 46198: Oozie - falcon integration properties not xconsidered on blueprint deployments

2016-04-14 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46198/ --- (Updated April 14, 2016, 3:19 p.m.) Review request for Ambari, Robert Levas,

Re: Review Request 46207: Owner of /tmp hdfs directory is yarn user on secure blueprint install

2016-04-14 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46207/#review128901 ---

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
> On April 14, 2016, 2:33 a.m., Matt wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/common.py, > > line 92 > > > > > > Are the expected return values just 0 and 1? > > >

Re: Review Request 46186: Changing HAWQ Ports through Ambari prevents HAWQ service from restarting

2016-04-14 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46186/#review128934 ---

Re: Review Request 46183: AMBARI-15876. Stack advisor work for adding and removing LLAP queue when Interactive query is turned on and off respectively.

2016-04-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46183/#review128936 --- Fix it, then Ship it!