Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Mugdha Varadkar
> On July 26, 2016, 3:35 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/KAFKA/kerberos.json, > > line 1 > > > > > > Is this change specific to HDP 2.5 or could some other stack

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Mugdha Varadkar
> On July 27, 2016, 4:12 a.m., Jayush Luniya wrote: > > Is Kerberized External Solr + Unkerberized Cluster a valid scenario? AFAIK, even if solr is kerberized all scripts will make calls using simple method. - Mugdha --- This is an

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Mugdha Varadkar
> On July 27, 2016, 4:15 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/kerberos.json, > > line 189 > > > > > > How did this typo not cause problems in the past?

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Mugdha Varadkar
> On July 26, 2016, 6:07 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/configuration/ranger-admin-site.xml, > > line 330 > > > > > > I thought this property will be

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/ --- (Updated July 27, 2016, 9:06 a.m.) Review request for Ambari, Alejandro

Re: Review Request 50470: AMBARI-17907 On AMS collector move the webapp address does not change

2016-07-27 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50470/#review143711 --- Ship it! Ship It! - Aleksandr Kovalenko On Июль 26, 2016,

Review Request 50489: Zeppelin install fails for non-root users due to wrong permissions

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

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/ --- (Updated Июль 27, 2016, 3:15 п.п.) Review request for Ambari, Dmitro

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-27 Thread Jonathan Hurley
> On July 26, 2016, 4:41 p.m., Robert Levas wrote: > > Committed to trunk > > ``` > > commit 124f48ef899fddb6bdb96ebea9aa3a6a1a6adbca > > Author: Anita Jebaraj > > Date: Tue Jul 26 16:25:30 2016 -0400 > > ``` > > > > Committed to branch-2.4 > > ``` > > commit

Re: Review Request 50489: Zeppelin install fails for non-root users due to wrong permissions

2016-07-27 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50489/#review143728 --- Ship it! Ship It! - Rohit Choudhary On July 27, 2016, 10:41

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/#review143745 ---

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/#review143743 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2016, 3:15

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/ --- (Updated Июль 27, 2016, 3:08 п.п.) Review request for Ambari, Dmitro

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Dmytro Sen
> On Июль 27, 2016, 3:03 п.п., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml, > > line 218 > > > > > > The common service definition should not have package

Review Request 50505: Coverity Scan Security Vulnerability - Resource Leak defects

2016-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50505/ --- Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Levas, and

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/#review143746 --- Ship it! Ship It! - Jayush Luniya On July 27, 2016, 9:06

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/#review143732 ---

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/#review143735 ---

Re: Review Request 50507: AMBARI-17886 : Disable INFO logs from HadoopMetrics2Reporter for hive

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50507/#review143741 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2016, 3:11

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

2016-07-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50479/#review143730 --- Ship it! Ship It! - Aravindan Vijayan On July 27, 2016,

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Robert Levas
> On July 26, 2016, 11:35 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json, > > lines 18-22 > > > > > > it seems like these should be set in a Jinja2 template

Re: Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Dmytro Sen
> On Июль 27, 2016, 3:10 п.п., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml, > > line 246 > > > > > > Can we fix all occurrences? Two more. oops. uploaded

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

2016-07-27 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50479/#review143768 --- Ship it! Ship It! - Sriharsha Chintalapani On July 27,

Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/ --- Review request for Ambari, bikassaha, Saisai Shao, and Sumit Mohanty. Bugs:

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Jayush Luniya
> On July 26, 2016, 3:35 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/HDFS/kerberos.json, > > line 1 > > > > > > Is this change specific to HDP 2.5 or could some other stack

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/#review143758 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2016, 5:10

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/#review143765 --- Ship it! Ship It! - Jayush Luniya On July 27, 2016, 5:10

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

2016-07-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50431/#review143774 --- Ship it! Ship It! - Alejandro Fernandez On July 27, 2016,

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread Robert Levas
> On July 27, 2016, 2:35 p.m., bikassaha wrote: > > I think the Spark and Spark2 design assumes the same keytab and user for > > Spark and Spark2 so that the transition between the two and concurrent use > > of both is seamless. So HDFS files, e.g for ATS data from both apps should > > be

Review Request 50526: NameNode High Availability Health Alert Issue

2016-07-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50526/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate

Re: Review Request 50529: Kafka brokers went down after Ambari upgrade due to IllegalArgumentException

2016-07-27 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50529/#review143796 --- Ship it! Ship It! - Sriharsha Chintalapani On July 27,

Re: Review Request 50526: NameNode High Availability Health Alert Issue

2016-07-27 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50526/#review143789 ---

Re: Review Request 50526: NameNode High Availability Health Alert Issue

2016-07-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50526/#review143785 --- Ship it! Ship It! - Nate Cole On July 27, 2016, 3:06 p.m.,

Review Request 50529: Kafka brokers went down after Ambari upgrade due to IllegalArgumentException

2016-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50529/ --- Review request for Ambari, Robert Levas and Sumit Mohanty. Bugs: AMBARI-17929

Review Request 50531: AMBARI-17931. Enable log4j compression for HiveMetastore, HiveServer2 and Hive Server Interactive.

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

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

2016-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/#review143779 --- Ship it! Ship It! - Robert Levas On July 27, 2016, 5:06

Re: Review Request 50526: NameNode High Availability Health Alert Issue

2016-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50526/#review143781 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 27, 2016,

Re: Review Request 50529: Kafka brokers went down after Ambari upgrade due to IllegalArgumentException

2016-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50529/#review143794 --- Ship it! Ship It!

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread bikas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/#review143801 ---

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

2016-07-27 Thread Alejandro Fernandez
> On July 27, 2016, 6:37 p.m., Alejandro Fernandez wrote: > > Ship It! Pushed to trunk, commit f428a6b97a52f6dee2e1cf7c78327770fd119ae6 branch-2.4, commit c91d18cac42bacab5d5a3c7745a6339172e2 - Alejandro --- This is an

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread bikas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/#review143771 --- I think the Spark and Spark2 design assumes the same keytab and

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

2016-07-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50431/#review143772 ---

Re: Review Request 50476: UI does not allow to delete service is some host components are in INSTALLED_FAILED state

2016-07-27 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50476/ --- (Updated July 27, 2016, 7:07 p.m.) Review request for Ambari, Zhe (Joe) Wang,

Re: Review Request 50500: Request for HBase metrics making a call to Namenode and Nimbus for that metric

2016-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50500/#review143788 --- Ship it! Ship It! - Sid Wagle On July 27, 2016, 11:44 a.m.,

Re: Review Request 50531: AMBARI-17931. Enable log4j compression for HiveMetastore, HiveServer2 and Hive Server Interactive.

2016-07-27 Thread Sushanth Sowmyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50531/#review143806 --- Ship it! Looks good to me, +1 - Sushanth Sowmyan On July

Review Request 50532: [Preview] Disable INFO logs from HadoopMetrics2Reporter for hive on upgrade to 2.5

2016-07-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50532/ --- Review request for Ambari, Alejandro Fernandez, Dmytro Grinenko, Jonathan

Re: Review Request 50531: AMBARI-17931. Enable log4j compression for HiveMetastore, HiveServer2 and Hive Server Interactive.

2016-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50531/#review143824 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2016, 8:27

Review Request 50506: HDFS Service check failure on wireencrypted clusters:Ambari2400+HDP-2.4.2.0

2016-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50506/ --- Review request for Ambari, Dmitro Lisnichenko, Sumit Mohanty, and Vitalyi

Review Request 50545: Ambari should not recursively chown for HAWQ hdfs upon every start

2016-07-27 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50545/ --- Review request for Ambari, Alexander Denissov and Matt. Bugs: AMBARI-17938

Review Request 50547: AMBARI-17939: Validations are missing for HDFS properties recommended by HAWQ

2016-07-27 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50547/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, Lav Jain, and

Re: Review Request 50539: EU to HDP 2.5 failed since config type 'sqoop-atlas-application.properties' has not been created

2016-07-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50539/#review143873 --- Ship it! Ship It! - Tim Thorpe On July 27, 2016, 11:06

Re: Review Request 50539: EU to HDP 2.5 failed since config type 'sqoop-atlas-application.properties' has not been created

2016-07-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50539/#review143872 ---

Review Request 50542: AMBARI-17936 Log search tab seems to be visible for all user roles

2016-07-27 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50542/ --- Review request for Ambari, Alexandr Antonenko, Jaimin Jetly, Richard Zang,

Review Request 50544: Ambari install/init should create a new gpadmin database

2016-07-27 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50544/ --- Review request for Ambari, Alexander Denissov and Matt. Bugs: AMBARI-17937

Re: Review Request 50544: Ambari install/init should create a new gpadmin database

2016-07-27 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50544/#review143868 --- Ship it! Ship It! - Alexander Denissov On July 27, 2016,

Re: Review Request 50512: Spark and Spark2 should use different keytab files to avoid ACL issues

2016-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50512/ --- (Updated July 27, 2016, 6:10 p.m.) Review request for Ambari, bikassaha,

Review Request 50539: EU to HDP 2.5 failed since config type 'sqoop-atlas-application.properties' has not been created

2016-07-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50539/ --- Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, Jonathan

Review Request 50500: Request for HBase metrics making a call to Namenode and Nimbus for that metric

2016-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50500/ --- Review request for Ambari, Aravindan Vijayan, Myroslav Papirkovskyy, and Sid

Re: Review Request 50489: Zeppelin install fails for non-root users due to wrong permissions

2016-07-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50489/#review143719 --- Ship it! Ship It! - Gaurav Nagar On July 27, 2016, 10:41

Re: Review Request 50489: Zeppelin install fails for non-root users due to wrong permissions

2016-07-27 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50489/#review143715 --- Ship it! Ship It! - Prabhjyot Singh On July 27, 2016, 10:41

Re: Review Request 50545: Ambari should not recursively chown for HAWQ hdfs upon every start

2016-07-27 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50545/#review143884 --- Fix it, then Ship it! Check impact on unit test. We have an

Re: Review Request 50544: Ambari install/init should create a new gpadmin database

2016-07-27 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50544/#review143885 --- Ship it! Ship It! - Matt On July 27, 2016, 4:41 p.m., Lav

Re: Review Request 50475: AMBARI-17778 Add usage information in ambari-server script

2016-07-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50475/#review143727 --- Ship it! Ship It! - Nate Cole On July 26, 2016, 8:30 p.m.,

Re: Review Request 50489: Zeppelin install fails for non-root users due to wrong permissions

2016-07-27 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50489/#review143720 --- Ship it! Ship It! - Nitiraj Rathore On July 27, 2016, 10:41