Review Request 56298: AMBARI-19871 - Config version switch/compare/revert doesn't work

2017-02-03 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56298/ --- Review request for Ambari, Andrii Tkach, Xi Wang, and Yusaku Sako. Bugs: AMBARI

Re: Review Request 56278: AMBARI-19864 : Zookeeper namespace for Hive service should be taken from hive-interactive-site, when only Hive-server Interactive is installed.

2017-02-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56278/#review164233 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

Re: Review Request 56278: AMBARI-19864 : Zookeeper namespace for Hive service should be taken from hive-interactive-site, when only Hive-server Interactive is installed.

2017-02-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56278/#review164234 --- Ship it! Fix it and ship it. - Sumit Mohanty On Feb. 3, 2017

Re: Review Request 56189: AMBARI-19827. HiveServer2 Interactive won't start in clusters with less memory.

2017-02-03 Thread Swapan Shridhar
> On Feb. 2, 2017, 3:19 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py, > > line 931 > > > > > > Lets add a test that goes through this condition. This one h

Re: Review Request 56189: AMBARI-19827. HiveServer2 Interactive won't start in clusters with less memory.

2017-02-03 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56189/ --- (Updated Feb. 4, 2017, 1 a.m.) Review request for Ambari, Alejandro Fernandez a

Re: Review Request 56189: AMBARI-19827. HiveServer2 Interactive won't start in clusters with less memory.

2017-02-03 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56189/#review164214 --- ambari-server/src/test/python/stacks/2.5/common/test_stack_adviso

Re: Review Request 56266: The user must be clearly communicated about YARN pre-emption requirements when Hive LLAP is enabled

2017-02-03 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56266/#review164185 --- Ship it! Ship It! - Yusaku Sako On Feb. 3, 2017, 3:44 a.m.,

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-03 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/#review164179 --- Ship it! Ship It! - Sid Wagle On Feb. 3, 2017, 9:31 p.m., Al

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/ --- (Updated Feb. 3, 2017, 9:31 p.m.) Review request for Ambari, Dmytro Sen, Sid Wa

Re: Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-03 Thread Amruta Borkar
> On Feb. 3, 2017, 1:28 p.m., Di Li wrote: > > Ship It! > > Di Li wrote: > pushed to trunk as > https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=ecfc951b5c8589b9225684c3ad2660cdb4ef4445 Thank you Di. - Amruta --- Th

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/#review164148 --- ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/#review164145 --- Ship it! Ship It! - Robert Nettleton On Feb. 3, 2017, 4:10 p

Re: Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-03 Thread Di Li
> On Feb. 3, 2017, 1:28 p.m., Di Li wrote: > > Ship It! pushed to trunk as https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=ecfc951b5c8589b9225684c3ad2660cdb4ef4445 - Di --- This is an automatically generated e-mail. To

Re: Review Request 56124: AMBARI-19795 Ambari upgrade to not add ranger plugin configs under ranger plugin supported services

2017-02-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56124/#review164136 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 31, 2017, 1:

Re: Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56235/#review164135 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 2, 2017, 8:0

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/ --- (Updated Feb. 3, 2017, 5:49 p.m.) Review request for Ambari, Miklos Gergely, Ro

Re: Review Request 56264: Add "live_hosts" metric in AMS for apps

2017-02-03 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56264/ --- (Updated Feb. 3, 2017, 5:22 p.m.) Review request for Ambari, Aravindan Vijayan,

Re: Review Request 56282: AMBARI-19867. Grafana install fails if custom directory's parent does not exist

2017-02-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56282/#review164129 --- Ship it! Ship It! - Sebastian Toader On Feb. 3, 2017, 6:13 p

Re: Review Request 56282: AMBARI-19867. Grafana install fails if custom directory's parent does not exist

2017-02-03 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56282/#review164128 --- Ship it! Ship It! - Aravindan Vijayan On Feb. 3, 2017, 5:13

Review Request 56282: AMBARI-19867. Grafana install fails if custom directory's parent does not exist

2017-02-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56282/ --- Review request for Ambari, Aravindan Vijayan, Sebastian Toader, and Sid Wagle.

Re: Review Request 56281: AMBARI-19865: Ambari-server logon failure: Hadoop-common dependency on servlet-api 2.5

2017-02-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56281/ --- (Updated Feb. 3, 2017, 9:12 a.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 56264: Add "live_hosts" metric in AMS for apps

2017-02-03 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56264/ --- (Updated Feb. 3, 2017, 5:09 p.m.) Review request for Ambari, Aravindan Vijayan,

Re: Review Request 56264: Add "live_hosts" metric in AMS for apps

2017-02-03 Thread Sid Wagle
> On Feb. 3, 2017, 4:47 p.m., Aravindan Vijayan wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricClusterAggregatorSecond.java, > > line 195 > >

Re: Review Request 56264: Add "live_hosts" metric in AMS for apps

2017-02-03 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56264/#review164125 --- ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/a

Re: Review Request 56281: AMBARI-19865: Ambari-server logon failure: Hadoop-common dependency on servlet-api 2.5

2017-02-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56281/#review164124 --- Ship it! Ship It! - Sumit Mohanty On Feb. 3, 2017, 4:15 p.m.

Re: Review Request 55989: Perf: start/stop all actions works much slower after few days of testing

2017-02-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55989/#review164123 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Re: Review Request 56281: AMBARI-19865: Ambari-server logon failure: Hadoop-common dependency on servlet-api 2.5

2017-02-03 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56281/#review164122 --- Ship it! Ship It! - Attila Magyar On Feb. 3, 2017, 4:15 p.m.

Review Request 56281: AMBARI-19865: Ambari-server logon failure: Hadoop-common dependency on servlet-api 2.5

2017-02-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56281/ --- Review request for Ambari, Attila Doroszlai, Attila Magyar, and Sumit Mohanty.

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/ --- (Updated Feb. 3, 2017, 4:10 p.m.) Review request for Ambari, Oliver Szabo and R

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/ --- (Updated Feb. 3, 2017, 3:23 p.m.) Review request for Ambari, Oliver Szabo and R

Re: Review Request 56230: Stack advisor issues encountered

2017-02-03 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56230/ --- (Updated Фев. 3, 2017, 3:13 п.п.) Review request for Ambari, Aravindan Vijayan,

Review Request 56278: AMBARI-19864 : Zookeeper namespace for Hive service should be taken from hive-interactive-site, when only Hive-server Interactive is installed.

2017-02-03 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56278/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Mugdha Varadkar, O

Re: Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-03 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56235/#review164118 --- Ship it! Ship It! - Di Li On Feb. 2, 2017, 8:01 p.m., Amruta

Review Request 56276: AMBARI-19845 Secure Ranger passwords in Ambari Stacks

2017-02-03 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56276/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, and

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/#review164116 --- Ship it! Ship It! - Oliver Szabo On Feb. 3, 2017, 10:17 a.m.

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-03 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review164113 --- Ship it! Ship It! - Dmitro Lisnichenko On Feb. 3, 2017, 12:3

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review164110 --- Ship it! Ship It! - Sebastian Toader On Feb. 3, 2017, 11:39

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/ --- (Updated Feb. 3, 2017, 11:39 a.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/ --- (Updated Feb. 3, 2017, 10:17 a.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 56264: Add "live_hosts" metric in AMS for apps

2017-02-03 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56264/#review164108 --- Ship it! Ship It! - Dmytro Sen On Фев. 3, 2017, 3:17 д.п., S

Re: Review Request 56273: JS error on host configs page

2017-02-03 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56273/#review164106 --- Ship it! Ship It! - Denys Buzhor On Feb. 3, 2017, 8:49 a.m.,

Review Request 56274: Fix Log Search User Config bugs

2017-02-03 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs: AMBARI-1986

Re: Review Request 55099: Disable auto start before RU/EU and enable during finalization phase

2017-02-03 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55099/#review164104 --- Ship it! Ship It! - Aleksandr Kovalenko On Фев. 2, 2017, 7:1

Re: Review Request 55099: Disable auto start before RU/EU and enable during finalization phase

2017-02-03 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55099/#review164105 --- Ship it! Ship It! - Aleksandr Kovalenko On Фев. 2, 2017, 7:1

Review Request 56273: JS error on host configs page

2017-02-03 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56273/ --- Review request for Ambari and Denys Buzhor. Bugs: ambari-19862 https://issu