Re: Review Request 48569: On Enabling HS Interactive default queue shown for llap is 'default' queue

2016-06-10 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48569/#review137090 --- Ship it! Ship It! - Zhe (Joe) Wang On June 11, 2016, 12:07

Review Request 48569: On Enabling HS Interactive default queue shown for llap is 'default' queue

2016-06-10 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48569/ --- Review request for Ambari, Zhe (Joe) Wang, Srimanth Gunturi, and Yusaku Sako.

Review Request 48568: AMBARI-17176 VDF: include default version definition in list, even if Internet Access is available

2016-06-10 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48568/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.

Re: Review Request 48395: AMBARI-17027: Metrics Collector API: Introduce basic series aggregation functions

2016-06-10 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48395/#review137086 ---

Re: Review Request 48532: AMBARI-17157 Storm 1.0 log4j config update

2016-06-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48532/#review137082 --- Ship it! Ship It! - Alejandro Fernandez On June 10, 2016,

Re: Review Request 48547: AMBARI-17164 Handle Ranger upgrade scenario in Kerberized env

2016-06-10 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48547/#review137079 --- Ship it! Ship It! - Srimanth Gunturi On June 10, 2016,

Re: Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Alejandro Fernandez
> On June 10, 2016, 8:07 p.m., Sowmya Ramesh wrote: > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py, > > line 147 > > > > > > I think path before was

Re: Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48561/ --- (Updated June 10, 2016, 9 p.m.) Review request for Ambari, Di Li, Dmitro

Re: Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Sowmya Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48561/#review137067 ---

Re: Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48561/#review137066 --- Ship it! Ship It! - Tim Thorpe On June 10, 2016, 6:13 p.m.,

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-10 Thread Don Bosco Durai
> On June 10, 2016, 6:08 p.m., Miklos Gergely wrote: > > Good feedback. Let me do this before we commit. - Don Bosco --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/#review137048

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-10 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/ --- (Updated June 10, 2016, 7:39 p.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 48553: Blueprint processor should handle manual stack definition changes without failing deployment

2016-06-10 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48553/#review137060 --- Ship it! In the context of this issue the fix looks good. As

Re: Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48561/#review137059 --- Fix it, then Ship it!

Re: Review Request 48562: Allow option to skip duplicate URL checking when creating VDF

2016-06-10 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48562/ --- (Updated June 10, 2016, 3:14 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48553: Blueprint processor should handle manual stack definition changes without failing deployment

2016-06-10 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48553/#review137057 --- Ship it! Ship It! - Robert Levas On June 10, 2016, 3:09

Re: Review Request 48562: Allow option to skip duplicate URL checking when creating VDF

2016-06-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48562/#review137056 --- Ship it! Ship It! - Jonathan Hurley On June 10, 2016, 2:26

Re: Review Request 48562: Allow option to skip duplicate URL checking when creating VDF

2016-06-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48562/#review137052 --- Ship it! Ship It! - Alejandro Fernandez On June 10, 2016,

Review Request 48562: Allow option to skip duplicate URL checking when creating VDF

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

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-10 Thread Oliver Szabo
> On June 9, 2016, 6:14 p.m., Oliver Szabo wrote: > > ambari-logsearch/ambari-logsearch-portal/src/main/configsets/hadoop_logs/conf/solrconfig.xml.j2, > > line 661 > > > > > > can you add these changes to the stack

Review Request 48561: Falcon to create data-mirroring directory in HDFS if extensions is supported

2016-06-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48561/ --- Review request for Ambari, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Jayush

Re: Review Request 48405: [Log Search portal ]Date range on comparison tab are different from date range of selected logs for comparison

2016-06-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48405/#review137049 --- Ship it! Ship It! - Miklos Gergely On June 8, 2016, 10:27

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/#review137048 ---

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-10 Thread Nate Cole
> On June 10, 2016, 1:40 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml, > > line 889 > > > > > > hosts="all" is the default, you can remove

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-10 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/ --- (Updated June 10, 2016, 5:53 p.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-10 Thread Don Bosco Durai
> On June 9, 2016, 6:14 p.m., Oliver Szabo wrote: > > I think, we don't need this to be configurable. Also, do I need to modify any other file other than the .j2? I thought, we were reusing the same files - Don Bosco --- This is an

Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

2016-06-10 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/#review137043 --- There should be some java tests for this new functionality. -

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48549/#review137042 ---

Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

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

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/#review137019 --- Ship it! Ship It! - Sebastian Toader On June 10, 2016, 5:08

Re: Review Request 48553: Blueprint processor should handle manual stack definition changes without failing deployment

2016-06-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48553/ --- (Updated June 10, 2016, 3:39 p.m.) Review request for Ambari, Laszlo Puskas,

Re: Review Request 48555: Kerberization of a cluster deletes ATS service

2016-06-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48555/#review137014 --- Ship it! Ship It! - Robert Nettleton On June 10, 2016, 2:51

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/#review137013 --- Ship it! Ship It! - Robert Nettleton On June 10, 2016, 3:08

Re: Review Request 48414: AMBARI-17118 Incorrect formated external url in ranger configuration - causes Namenode startup failure

2016-06-10 Thread Velmurugan Periasamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48414/#review137012 --- Ship it! Ship It! - Velmurugan Periasamy On June 10, 2016,

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

2016-06-10 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/#review137011 ---

Re: Review Request 48547: AMBARI-17164 Handle Ranger upgrade scenario in Kerberized env

2016-06-10 Thread Velmurugan Periasamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48547/#review137009 --- Ship it! Ship It! - Velmurugan Periasamy On June 10, 2016,

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-10 Thread Velmurugan Periasamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48549/#review137008 --- Ship it! Ship It! - Velmurugan Periasamy On June 10, 2016,

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/ --- (Updated June 10, 2016, 3:08 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/ --- (Updated June 10, 2016, 3 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48555: Kerberization of a cluster deletes ATS service

2016-06-10 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48555/#review137003 --- Ship it! Ship It! - Sumit Mohanty On June 10, 2016, 2:51

Review Request 48555: Kerberization of a cluster deletes ATS service

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

Re: Review Request 48480: View Config- views created for 2nd remote cluster points to 1st remote cluster

2016-06-10 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48480/#review136995 --- Ship it! Ship It! - Nitiraj Rathore On June 9, 2016, 8:43

Review Request 48553: Blueprint processor should handle manual stack definition changes without failing deployment

2016-06-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48553/ --- Review request for Ambari, Laszlo Puskas, Mahadev Konar, Oliver Szabo, Sandor

Re: Review Request 48414: AMBARI-17118 Incorrect formated external url in ranger configuration - causes Namenode startup failure

2016-06-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48414/ --- (Updated June 10, 2016, 1:30 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/#review136994 ---

Re: Review Request 48547: AMBARI-17164 Handle Ranger upgrade scenario in Kerberized env

2016-06-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48547/ --- (Updated June 10, 2016, 12:50 p.m.) Review request for Ambari, Gautam Borad,

Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48549/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Nate Cole, Srimanth

Review Request 48547: Handle Ranger upgrade scenario in Kerberized env

2016-06-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48547/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Nate Cole, Srimanth

Re: Review Request 48404: Add user home directory verification as part of Service Check

2016-06-10 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48404/ --- (Updated June 10, 2016, 12:39 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 48516: SPNEGO keytab and principal configuration for HBase web UIs

2016-06-10 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48516/#review136991 --- Ship it! Ship It! - Robert Levas On June 9, 2016, 7 p.m.,

Review Request 48545: View config- Allows remote cluster creation using a user having no permissions on the cluster

2016-06-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48545/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav

Re: Review Request 48546: Wildcards shouldn't be used for package installation

2016-06-10 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48546/#review136990 --- Ship it! Ship It! - Dmitro Lisnichenko On June 10, 2016,

Re: Review Request 48546: Wildcards shouldn't be used for package installation

2016-06-10 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48546/ --- (Updated June 10, 2016, 12:13 p.m.) Review request for Ambari and Dmitro

Review Request 48546: Xml manifests should not include wildcards but full versions for package-version

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

Re: Review Request 48516: SPNEGO keytab and principal configuration for HBase web UIs

2016-06-10 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48516/#review136989 --- Ship it! Ship It! - Nate Cole On June 9, 2016, 7 p.m., Josh

Re: Review Request 48398: Capacity scheduler view - problem with queue percent rounding

2016-06-10 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48398/#review136986 --- Ship it! Ship It! - Sreenath Somarajapuram On June 10,

Review Request 48540: Ubuntu 16, Hive Metastore Start failed

2016-06-10 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48540/ --- Review request for Ambari and Myroslav Papirkovskyy. Bugs: AMBARI-17161

Re: Review Request 48398: Capacity scheduler view - problem with queue percent rounding

2016-06-10 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48398/ --- (Updated June 10, 2016, 9:58 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-10 Thread Dmytro Sen
> On Июнь 9, 2016, 4:05 п.п., Daniel Gergely wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Please consider

Review Request 48414: AMBARI-17118 Incorrect formated external url in ranger configuration - causes Namenode startup failure

2016-06-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48414/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Srimanth Gunturi,

Re: Review Request 48501: AMBARI-17147: OSError: [Errno 1] Operation not permitted with non-root ambari agent user

2016-06-10 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48501/#review136982 --- Ship it! Ship It! - Sebastian Toader On June 9, 2016, 8:56

Re: Review Request 48480: View Config- views created for 2nd remote cluster points to 1st remote cluster

2016-06-10 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48480/#review136980 --- Ship it! Ship It! - Pallav Kulshreshtha On June 9, 2016,

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-10 Thread Daniel Gergely
> On jún. 9, 2016, 4:05 du, Daniel Gergely wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Please consider

Re: Review Request 48508: AMBARI-17149 : HadoopTimelineMetricsSink from AMS prevents HBase RS process shutdown

2016-06-10 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48508/#review136973 --- Ship it! Ship It! - Dmytro Sen On Июнь 10, 2016, полночь,