Re: Review Request 53959: NPE when authenticating via a Centrify LDAP proxy

2016-11-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53959/#review156553 ---

Re: Review Request 53959: NPE when authenticating via a Centrify LDAP proxy

2016-11-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53959/#review156554 --- Ship it! Ship It! - Sebastian Toader On Nov. 21, 2016, 7:43

Re: Review Request 53972: AMBARI-18955. Cleanup 'LLAP package' folders before a given Hive Server Interactive restart.

2016-11-21 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53972/#review156535 --- Ship it! Ship It! - Sumit Mohanty On Nov. 22, 2016, 12:59

Re: Review Request 53972: AMBARI-18955. Cleanup 'LLAP package' folders before a given Hive Server Interactive restart.

2016-11-21 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53972/ --- (Updated Nov. 22, 2016, 12:59 a.m.) Review request for Ambari, Alejandro

Re: Review Request 53972: AMBARI-18955. Cleanup 'LLAP package' folders before a given Hive Server Interactive restart.

2016-11-21 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53972/#review156534 --- Ship it! Ship It! - Sumit Mohanty On Nov. 22, 2016, 12:50

Review Request 53972: AMBARI-18955. Cleanup 'LLAP package' folders before a given Hive Server Interactive restart.

2016-11-21 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53972/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 53969: When HS2Interactive is enabled, any hive config change results in a CS queue refresh

2016-11-21 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53969/ --- (Updated Nov. 21, 2016, 11:13 p.m.) Review request for Ambari, Jaimin Jetly,

Review Request 53969: When HS2Interactive is enabled, any hive config change results in a CS queue refresh

2016-11-21 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53969/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.

Re: Review Request 53922: AMBARI-18942 - Auto-start services: Avoid full page reload after "Save" / "Discard"

2016-11-21 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53922/#review156508 --- Ship it! Ship It! - Yusaku Sako On Nov. 19, 2016, 3:10

Re: Review Request 53801: AMBARI-18901. Use 'Number of LLAP Nodes' selected as the driver for LLAP config calculations.

2016-11-21 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53801/ --- (Updated Nov. 21, 2016, 8:12 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53958: AMBARI-18934 - Management packs should be able to link grafana dashboards and service metrics for custom services

2016-11-21 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53958/#review156502 --- Ship it! Ship It! - Di Li On Nov. 21, 2016, 6:53 p.m., Tim

Re: Review Request 53959: NPE when authenticating via a Centrify LDAP proxy

2016-11-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53959/#review156495 --- Ship it! Ship It! - Oliver Szabo On Nov. 21, 2016, 6:43

Re: Review Request 53904: Implement Create Alerts: step 1 select alert type

2016-11-21 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53904/#review156492 --- Ship it! Ship It! - Vivek Ratnavel Subramanian On Nov. 18,

Re: Review Request 53629: AMBARI-18836 Remove group readable from hdfs headless keytab

2016-11-21 Thread Robert Levas
> On Nov. 11, 2016, 5:28 a.m., Robert Levas wrote: > > Ship It! > > Shi Wang wrote: > Hi Robert, could you help commit it if there is no other concern? Committed... ``` commit 38076327525986b780942f33eff01d2de4a70ce2 Author: Shi Wang Date: Mon Nov 21 14:03:32 2016

Re: Review Request 52155: ambari server upgrade ambari to 2.1.1 duplicate key error

2016-11-21 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52155/#review156487 --- Ship it! Ship It! - Alejandro Fernandez On Nov. 21, 2016,

Re: Review Request 53958: AMBARI-18934 - Management packs should be able to link grafana dashboards and service metrics for custom services

2016-11-21 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53958/ --- (Updated Nov. 21, 2016, 6:53 p.m.) Review request for Ambari, Di Li and Jayush

Re: Review Request 53958: AMBARI-18934 - Management packs should be able to link grafana dashboards and service metrics for custom services

2016-11-21 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53958/ --- (Updated Nov. 21, 2016, 6:51 p.m.) Review request for Ambari, Di Li and Jayush

Re: Review Request 53801: AMBARI-18901. Use 'Number of LLAP Nodes' selected as the driver for LLAP config calculations.

2016-11-21 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53801/#review156486 --- Ship it! Ship It! - Alejandro Fernandez On Nov. 19, 2016,

Re: Review Request 53722: AMBARI-18425:Support PAM as an authentication option for Ranger in Ambari

2016-11-21 Thread Shi Wang
> On Nov. 21, 2016, 5:17 a.m., Mugdha Varadkar wrote: > > Ship It! > > Mugdha Varadkar wrote: > Hi Shi Wang, Please attach latest updated patch on Apache Jira > AMBARI-18425 also. > > Shi Wang wrote: > Thanks Mugdha, I just updated it to the latest version. Hi Mugdha, Could you help

Re: Review Request 53629: AMBARI-18836 Remove group readable from hdfs headless keytab

2016-11-21 Thread Shi Wang
> On Nov. 11, 2016, 10:28 a.m., Robert Levas wrote: > > Ship It! Hi Robert, could you help commit it if there is no other concern? - Shi --- This is an automatically generated e-mail. To reply, visit:

Review Request 53959: NPE when authenticating via a Centrify LDAP proxy

2016-11-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53959/ --- Review request for Ambari, Laszlo Puskas, Oliver Szabo, and Sebastian Toader.

Review Request 53958: AMBARI-18934 - Management packs should be able to link grafana dashboards and service metrics for custom services

2016-11-21 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53958/ --- Review request for Ambari, Di Li and Jayush Luniya. Bugs: AMBARI-18934

Re: Review Request 53722: AMBARI-18425:Support PAM as an authentication option for Ranger in Ambari

2016-11-21 Thread Shi Wang
> On Nov. 21, 2016, 5:17 a.m., Mugdha Varadkar wrote: > > Ship It! > > Mugdha Varadkar wrote: > Hi Shi Wang, Please attach latest updated patch on Apache Jira > AMBARI-18425 also. Thanks Mugdha, I just updated it to the latest version. - Shi

Re: Review Request 53901: Allow generic Service Upgrade Packs for all targets

2016-11-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53901/ --- (Updated Nov. 21, 2016, 1:11 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53951: Refactor hive() function in HIVE service

2016-11-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53951/#review156465 --- Ship it! Ship It! - Dmitro Lisnichenko On Nov. 21, 2016, 6

Review Request 53951: Refactor hive() function in HIVE service

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

Re: Review Request 53947: hive metrics : support hs2 with remote metastore

2016-11-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53947/#review156457 --- Ship it! Ship It! - Dmitro Lisnichenko On Nov. 21, 2016,

Review Request 53947: hive metrics : support hs2 with remote metastore

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

Re: Review Request 53901: Allow generic Service Upgrade Packs for all targets

2016-11-21 Thread Nate Cole
> On Nov. 18, 2016, 4:56 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java, > > lines 912-913 > > > > > > Is this really a warning? It's not a problem if

Re: Review Request 53945: Infinite spinner displayed instead of some Flume Agent graphs

2016-11-21 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53945/#review156454 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лис. 21, 2016,

Review Request 53945: Infinite spinner displayed instead of some Flume Agent graphs

2016-11-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53945/ --- Review request for Ambari, Aravind Gottipati, Sid Wagle, and Vitalyi Brodetskyi.

Review Request 53944: Ambari Integration for Zeppelin and Spark 2.0

2016-11-21 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53944/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar,

Re: Review Request 53914: Perf: Fix bugs in deploy-gce-perf-cluster.py to generate correct config file

2016-11-21 Thread Vitalyi Brodetskyi
> On Лис. 21, 2016, 8:13 до полудня, Dmitro Lisnichenko wrote: > > contrib/utils/perf/deploy-gce-perf-cluster.py, line 227 > > > > > > why do we need this sleep? because we are not able to ssh to cluster hosts

Re: Review Request 53843: Zeppelin view doesn't work with JDK 1.8_91+

2016-11-21 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53843/#review156451 --- Ship it! Ship It! - Pallav Kulshreshtha On Nov. 17, 2016,

Re: Review Request 53843: Zeppelin view doesn't work with JDK 1.8_91+

2016-11-21 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53843/#review156450 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Nov. 17, 2016,

Re: Review Request 53722: AMBARI-18425:Support PAM as an authentication option for Ranger in Ambari

2016-11-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53722/#review156449 --- Ship it! Ship It! - Robert Levas On Nov. 17, 2016, 4:05

Re: Review Request 53854: Agent Auto Restart Doesn't Release Ping Port

2016-11-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53854/ --- (Updated Ноя. 21, 2016, 9:27 д.п.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 53914: Perf: Fix bugs in deploy-gce-perf-cluster.py to generate correct config file

2016-11-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53914/#review156445 --- Fix it, then Ship it!