Re: Review Request 48273: AMBARI-17054 : Configure Atlas Ranger Plugin

2016-06-08 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48273/ --- (Updated June 9, 2016, 6:19 a.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 48403: Fixed implementation of on-ambari-upgrade support. Patch 1 - change validation rules and available fields

2016-06-08 Thread Dmitro Lisnichenko
> On June 8, 2016, 10:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java, > > line 98 > > > > > > So I had thought that we decided to make these optional

Review Request 48460: Ambari Views logs should include information of view instance

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

Review Request 48454: AMBARI-17103. Incorporating the inheritance logic for "tez_hive2/tez-site.xml" using tez/tez-site.xml. Also, removing 'hive.tez.java.opts' config from hive2/hive-site.xml as it w

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

Review Request 48454: AMBARI-17103. Incorporating the inheritance logic for "tez_hive2/tez-site.xml" using tez/tez-site.xml. Also, removing 'hive.tez.java.opts' config from hive2/hive-site.xml as it w

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

Re: Review Request 48030: AMBARI-16946 Storm Metrics Sink has high chance to discard some datapoints

2016-06-08 Thread Jungtaek Lim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48030/ --- (Updated 6 9, 2016, 1:58 오전) Review request for Ambari, Aravindan Vijayan, Srih

Review Request 48456: AMBARI-17134 - Add support flag for user type 'JWT'

2016-06-08 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48456/ --- Review request for Ambari and Zhe (Joe) Wang. Bugs: AMBARI-17134 https://is

Re: Review Request 48451: AMBARI-17132 - Typing the whole component name vs selecting drop down filter in hosts page gives 2 different results

2016-06-08 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48451/#review136734 --- Ship it! Ship It! - Zhe (Joe) Wang On June 9, 2016, 12:09 a.

Review Request 48451: AMBARI-17132 - Typing the whole component name vs selecting drop down filter in hosts page gives 2 different results

2016-06-08 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48451/ --- Review request for Ambari and Zhe (Joe) Wang. Bugs: AMBARI-17132 https://is

Re: Review Request 48184: clean up import * for SPARK2 service scripts in common-services

2016-06-08 Thread Juanjo Marron
> On June 7, 2016, 6:45 p.m., Alejandro Fernandez wrote: > > Ship It! I dont have privileges to commit. Could someone deliver it to trunk so I can mark the JIRA https://issues.apache.org/jira/browse/AMBARI-16101 as complete, please? Thanks - Juanjo -

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Eugene Chekanskiy
> On June 8, 2016, 7:47 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog230.java, > > lines 347-351 > > > > > > If this is needed in UpgradeCatalog240, why is

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Eugene Chekanskiy
> On June 8, 2016, 7:47 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog230.java, > > lines 347-351 > > > > > > If this is needed in UpgradeCatalog240, why is

Re: Review Request 48439: App Timeline Server start fails on a kerberized cluster due to absence of hdfs keytab

2016-06-08 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48439/#review136685 --- Ship it! Ship It! - Robert Levas On June 8, 2016, 2:46 p.m.,

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48440/#review136677 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Re: Review Request 48096: AMBARI-16935: Retry and recover from component install failures

2016-06-08 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48096/ --- (Updated June 8, 2016, 12:22 p.m.) Review request for Ambari, Ajit Kumar, Jonat

Re: Review Request 48403: Fixed implementation of on-ambari-upgrade support. Patch 1 - change validation rules and available fields

2016-06-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48403/#review136681 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 48096: AMBARI-16935: Retry and recover from component install failures

2016-06-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48096/#review136679 --- Fix it, then Ship it! ambari-agent/src/main/python/ambari_agen

Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48440/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-17128 https://issu

Review Request 48439: App Timeline Server start fails on a kerberized cluster due to absence of hdfs keytab

2016-06-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48439/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-15871 https://issu

Review Request 48436: Ambari Stale Alert Triggers For Server-Side Performance Alert

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

Re: Review Request 48382: AMBARI-17101 Select Stack Page : "Next" button disabled for blank repo fields

2016-06-08 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48382/#review136669 --- Ship it! Ship It! - Jaimin Jetly On June 7, 2016, 11:22 p.m.

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

2016-06-08 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48404/#review136668 --- contrib/views/pig/pom.xml (line 146)

Re: Review Request 48332: Log search capability for Nifi

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48332/ --- (Updated June 8, 2016, 5:53 p.m.) Review request for Ambari, Jayush Luniya, Oli

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

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

Re: Review Request 48403: Fixed implementation of on-ambari-upgrade support. Patch 1 - change validation rules and available fields

2016-06-08 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48403/#review136659 --- ambari-server/src/main/java/org/apache/ambari/server/state/Proper

Re: Review Request 48419: Accumulo master start fails on non-root + umask 027

2016-06-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48419/#review136657 --- Ship it! Ship It! - Dmitro Lisnichenko On June 8, 2016, 7:07

Review Request 48419: Accumulo master start fails on non-root + umask 027

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

Re: Review Request 48273: AMBARI-17054 : Configure Atlas Ranger Plugin

2016-06-08 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48273/#review136656 --- Ship it! Ship It! - Srimanth Gunturi On June 6, 2016, 3:26 p

Re: Review Request 48416: Ambari warns the user about a bunch of rpm packages that are installed on barebone CentOS 7

2016-06-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48416/#review136646 --- Ship it! Ship It! - Dmitro Lisnichenko On June 8, 2016, 5:39

Re: Review Request 48415: Authorizations given to role-based principals must be dereferenced upon user login

2016-06-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48415/#review136645 --- Ship it! Ship It! - Jonathan Hurley On June 8, 2016, 9:53 a.

Review Request 48416: Ambari warns the user about a bunch of rpm packages that are installed on barebone CentOS 7

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

Re: Review Request 48415: Authorizations given to role-based principals must be dereferenced upon user login

2016-06-08 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48415/#review136644 --- Ship it! Ship It! - Nate Cole On June 8, 2016, 9:53 a.m., Ro

Re: Review Request 48413: Fix misnamed Zookeeper connect strings

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48413/ --- (Updated June 8, 2016, 2:14 p.m.) Review request for Ambari, Oliver Szabo, Robe

Review Request 48415: Authorizations given to role-based principals must be dereferenced upon user login

2016-06-08 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48415/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jonathan Hurley, Myroslav Papirkovs

Re: Review Request 48392: Deadlock While Updating Stale Configuration Cache During Upgrade

2016-06-08 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48392/#review136637 --- Ship it! Ship It! - Robert Levas On June 8, 2016, 8:44 a.m.,

Review Request 48413: Fix misnamed Zookeeper connect strings

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48413/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B

Re: Review Request 48392: Deadlock While Updating Stale Configuration Cache During Upgrade

2016-06-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48392/ --- (Updated June 8, 2016, 8:44 a.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 48392: Deadlock While Updating Stale Configuration Cache During Upgrade

2016-06-08 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48392/#review136627 --- Ship it! Ship It! - Nate Cole On June 7, 2016, 11:49 p.m., J

Re: Review Request 48410: Ambari-agent could not successfully restart on non-root after running as root

2016-06-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48410/#review136626 --- Ship it! Ship It! - Dmitro Lisnichenko On June 8, 2016, 3:17

Re: Review Request 48410: Ambari-agent could not successfully restart on non-root after running as root

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

Re: Review Request 48410: Ambari-agent could not successfully restart on non-root after running as root

2016-06-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48410/#review136625 --- Ship it! Ship It! - Dmitro Lisnichenko On June 8, 2016, 3:13

Re: Review Request 48410: Ambari-agent could not successfully restart on non-root after running as root

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

Re: Review Request 48410: Ambari-agent could not successfully restart on non-root after running as root

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

Review Request 48410: Heartbeat lost of ambari agent during install

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

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

2016-06-08 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48404/#review136624 --- Ship it! Ship It! - Nitiraj Rathore On June 8, 2016, 10:09 a

Re: Review Request 48147: Fix metric sink

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/ --- (Updated June 8, 2016, 11:16 a.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 48147: Fix metric sink + rename misnamed zk-connect-string variables.

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/ --- (Updated June 8, 2016, 11:09 a.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 48332: Log search capability for Nifi

2016-06-08 Thread Miklos Gergely
> On June 7, 2016, 1:20 p.m., Robert Nettleton wrote: > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py, > > line 233 > > > > > > Is NiFi being added to Ambari in 2.4?

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder on a newly installed cluster with log search

2016-06-08 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/#review136622 --- Ship it! Ship It! - Miklos Gergely On June 7, 2016, 4:06 p.m

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

2016-06-08 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48405/#review136621 --- Ship it! Ship It! - Oliver Szabo On June 8, 2016, 10:27 a.m.

Re: Review Request 48284: Retrieve specific metrics when Ambari queries NameNode HA states

2016-06-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48284/#review136620 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 8, 2016, 1

Re: Review Request 48284: Retrieve specific metrics when Ambari queries NameNode HA states

2016-06-08 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48284/ --- (Updated June 8, 2016, 10:32 a.m.) Review request for Ambari, Jonathan Hurley a

Re: Review Request 48407: [Change log directory prefix for service FLUME] Flume service not starting after changing the log directory

2016-06-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48407/#review136619 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 8, 2016, 1

Review Request 48407: [Change log directory prefix for service FLUME] Flume service not starting after changing the log directory

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

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

2016-06-08 Thread Dharmesh Makwana
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48405/ --- Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Don Bosco Durai

Re: Review Request 48325: Unit tests failing because of the order of hash sets/maps

2016-06-08 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48325/#review136618 --- Ship it! Ship It! - Oliver Szabo On June 7, 2016, 1:02 p.m.,

Re: Review Request 48406: DevDeploy: Ambari server is not started because of invalid config

2016-06-08 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48406/#review136617 --- Ship it! Ship It! - Andrew Onischuk On June 8, 2016, 10:12 a

Review Request 48406: DevDeploy: Ambari server is not started because of invalid config

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

Review Request 48403: Fixed implementation of on-ambari-upgrade support. Patch 1 - change validation rules and available fields

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

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

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

Re: Review Request 48279: Visual explain unit tests

2016-06-08 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48279/#review136616 --- Ship it! Ship It! - Rohit Choudhary On June 6, 2016, 2:24 p.

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-08 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48044/#review136615 --- ambari-common/src/main/python/resource_management/libraries/funct

Re: Review Request 48402: ATS needs a dynamic dependency on Tez and Spark

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

Review Request 48402: ATS needs a dynamic dependency on Tez and Spark

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

Re: Review Request 47941: [AMBARI-16920] Spark2 thrift server can not started due to miss of spark-thrift-fairscheduler.xml

2016-06-08 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/ --- (Updated June 8, 2016, 7:31 a.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 47941: [AMBARI-16920] Spark2 thrift server can not started due to miss of spark-thrift-fairscheduler.xml

2016-06-08 Thread Jeff Zhang
> On June 8, 2016, 1:08 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_service.py, > > line 61 > > > > > > This is calling what's meant to be