Re: Review Request 50207: Fix LogSearch Solr 'stop'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50207/#review142904 --- Ship it! Ship It! - Dharmesh Makwana On July 19, 2016,

Re: Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/#review142903 --- Ship it! Ship It! - Nahappan Somasundaram On July 19, 2016,

Re: Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/#review142896 --- Ship it! Ship It! - Swapan Shridhar On July 20, 2016, 2:01

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

> On July 19, 2016, 2:50 p.m., Lav Jain wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 392 > > > > > > Should config-name be set to hawq_rm_memory_limit_perseg?

Re: Review Request 50210: Add atlas.authentication.method.kerberos property to conf/atlas-application.properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50210/#review142891 --- Ship it! Ship It! - Sumit Mohanty On July 20, 2016, 2:07

Re: Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/#review142890 --- Ship it! Ship It! - Sumit Mohanty On July 20, 2016, 2:01

Re: Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/#review142889 ---

Review Request 50210: Add atlas.authentication.method.kerberos property to conf/atlas-application.properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50210/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Re: Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/#review142888 ---

Review Request 50219: Blueprint deployment configures "org.apache.atlas.hive.hook.HiveHook" twice for "hive.exec.post.hooks" config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50219/ --- Review request for Ambari, Dmytro Grinenko, Nahappan Somasundaram, Sumit

Re: Review Request 50207: Fix LogSearch Solr 'stop'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50207/#review142877 --- Ship it! Ship It! - Alejandro Fernandez On July 19, 2016,

Re: Review Request 50207: Fix LogSearch Solr 'stop'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50207/#review142874 --- Ship it! Ship It! - Oliver Szabo On July 19, 2016, 11:26

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

> On July 19, 2016, 1:59 p.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 213 > > > > > > this property has no impact on the logic inside

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

> On July 19, 2016, 2:50 p.m., Lav Jain wrote: > > ambari-server/src/test/python/common-services/HAWQ/test_service_advisor.py, > > line 378 > > > > > > Should line 378 be hosts["items"][2] instead of

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/ --- (Updated July 19, 2016, 4:34 p.m.) Review request for Ambari, Alexander

Review Request 50207: Fix LogSearch Solr 'stop'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50207/ --- Review request for Ambari, Hayat Behlim, Dharmesh Makwana, Oliver Szabo, Robert

Re: Review Request 50180: AMBARI-17784 AMS Storm Sink: remove redundant information from kafka offset metrics on storm-kafka (including AMBARI-17725)

> On 7 19, 2016, 7:29 오후, Aravindan Vijayan wrote: > > Jungtaek, Is the metricName generation logic same for both the legacy and > > current version of Storm? > > Sriharsha Chintalapani wrote: > Aravindan, Given that all these metrics coming in HDP-2.5 storm why are > we worried about

Re: Review Request 50203: AMS: Deleting ZK data in embedded mode results in external phoenix clients unable to connect on AMS restart

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50203/#review142856 --- Ship it! Ship It! - Aravindan Vijayan On July 19, 2016,

Re: Review Request 50178: AMBARI-17783: Add falcon to as oozie admin user for 2.5

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50178/#review142854 --- Pushed to trunk, commit 1ad5e59687232ae1491fd7bfa316030e347013e2

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/#review142847 ---

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/#review142849 --- Ship it! Ship It! - Lav Jain On July 19, 2016, 7:15 p.m.,

Re: Review Request 50180: AMBARI-17784 AMS Storm Sink: remove redundant information from kafka offset metrics on storm-kafka (including AMBARI-17725)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50180/#review142848 --- Ship it! Ship It! - Sriharsha Chintalapani On July 19,

Re: Review Request 50180: AMBARI-17784 AMS Storm Sink: remove redundant information from kafka offset metrics on storm-kafka (including AMBARI-17725)

> On July 19, 2016, 7:29 p.m., Aravindan Vijayan wrote: > > Jungtaek, Is the metricName generation logic same for both the legacy and > > current version of Storm? Aravindan, Given that all these metrics coming in HDP-2.5 storm why are we worried about legacy version. We are not even storing

Review Request 50203: AMS: Deleting ZK data in embedded mode results in external phoenix clients unable to connect on AMS restart

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50203/ --- Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sumit Mohanty.

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/#review142834 --- Ship it! Ship It! - Alexander Denissov On July 19, 2016,

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/#review142831 ---

Re: Review Request 50196: Ranger Admin Fails To Stop Or Start During Downgrade From HDP 2.5 to HDP 2.4

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50196/#review142832 --- Ship it! Ship It! - Nate Cole On July 19, 2016, 4:56 p.m.,

Re: Review Request 50196: Ranger Admin Fails To Stop Or Start During Downgrade From HDP 2.5 to HDP 2.4

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50196/ --- (Updated July 19, 2016, 4:56 p.m.) Review request for Ambari, Alejandro

Re: Review Request 50196: Ranger Admin Fails To Stop Or Start During Downgrade From HDP 2.5 to HDP 2.4

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50196/#review142824 --- Ship it! Ship It! - Jayush Luniya On July 19, 2016, 6:36

Re: Review Request 50087: Ambari should have a script to add new repository and service to existing stack

> On July 19, 2016, 12:05 p.m., Tim Thorpe wrote: > > I really like the ability to add new repos. I'm not so certain we need > > another way of adding services to an existing stack. > > > > Jayush and I already added new code to allow Ambari to add new stacks, > > extensions and custom

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

> On July 18, 2016, 10:58 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerberos.json, > > line 18 > > > > > > This is redundant... > > > >

Re: Review Request 50201: HiveServer port config change is missing in HDP-2.4 Upgrade Pack

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50201/#review142819 --- Ship it! Ship It! - Jonathan Hurley On July 19, 2016, 3:47

Re: Review Request 50178: AMBARI-17783: Add falcon to as oozie admin user for 2.5

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50178/#review142818 --- Ship it! Please take care of upgrade issue as Alejandro

Review Request 50201: HiveServer port config change is missing in HDP-2.4 Upgrade Pack

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50201/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

Re: Review Request 50178: AMBARI-17783: Add falcon to as oozie admin user for 2.5

> On July 19, 2016, 11:01 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py, > > line 35 > > > > > > This will affect only a brand new cluster install. >

Re: Review Request 50180: AMBARI-17784 AMS Storm Sink: remove redundant information from kafka offset metrics on storm-kafka (including AMBARI-17725)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50180/#review142812 --- Jungtaek, Is the metricName generation logic same for both the

Re: Review Request 50198: HAWQ fails to execute queries in low-memory environment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50198/ --- (Updated July 19, 2016, 12:15 p.m.) Review request for Ambari, Alexander

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

> On July 18, 2016, 6:58 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerberos.json, > > line 18 > > > > > > This is redundant... > > > >

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

> On July 18, 2016, 10:58 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerberos.json, > > line 18 > > > > > > This is redundant... > > > >

Re: Review Request 50165: AMBARI-17777 Unexpected behavior on Select Version page

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50165/#review142796 --- Ship it! Ship It! - Richard Zang On July 18, 2016, 10:20

Re: Review Request 50196: Ranger Admin Fails To Stop Or Start During Downgrade From HDP 2.5 to HDP 2.4

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50196/#review142792 --- Ship it! Ship It! - Nate Cole On July 19, 2016, 2:36 p.m.,

Review Request 50196: Ranger Admin Fails To Stop Or Start During Downgrade From HDP 2.5 to HDP 2.4

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50196/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Nate Cole.

Re: Review Request 50169: Atlas hooks for Hive, Falcon, Storm missing atlas.jaas.KafkaClient.option.principal & atlas.jaas.KafkaClient.option.keytab

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50169/#review142781 --- Ship it! Ship It! - Robert Levas On July 19, 2016, 1:50

Re: Review Request 50178: AMBARI-17783: Add falcon to as oozie admin user for 2.5

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50178/#review142779 ---

Re: Review Request 50195: Oozie Server Start is failed after upgrade from 2.2.2.0 to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50195/#review142777 --- Ship it! Ship It! - Alejandro Fernandez On July 19, 2016,

Re: Review Request 50195: Oozie Server Start is failed after upgrade from 2.2.2.0 to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50195/#review142772 --- Ship it! Ship It! - Myroslav Papirkovskyy On Липень 19,

Review Request 50195: Oozie Server Start is failed after upgrade from 2.2.2.0 to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50195/ --- Review request for Ambari, Dmytro Sen and Myroslav Papirkovskyy. Bugs:

Re: Review Request 50191: Spark service check fails on node with stopped Livy server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50191/ --- (Updated Июль 19, 2016, 4:53 п.п.) Review request for Ambari, Dmitro

Re: Review Request 50191: Spark service check fails on node with stopped Livy server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50191/#review142761 --- Ship it! Ship It! - Dmytro Grinenko On July 19, 2016, 4:15

Re: Review Request 50191: Spark service check fails on node with stopped Livy server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50191/#review142760 --- Ship it! Ship It! - Dmitro Lisnichenko On July 19, 2016,

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49891/#review142757 --- Ship it! Ship It! - Sid Wagle On July 19, 2016, 10:54 a.m.,

Re: Review Request 50140: AMBARI-17771 : Update alert definition for Ranger Admin after kerberos changes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50140/ --- (Updated July 19, 2016, 3:26 p.m.) Review request for Ambari, Alejandro

Re: Review Request 50069: Docker based dev environment for Log Search

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- (Updated July 19, 2016, 3:04 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 50069: Docker based dev environment for Log Search

> On July 19, 2016, 12:50 p.m., Nate Cole wrote: > > ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/LogFeederUtil.java, > > lines 401-403 > > > > > > This entire class needs

Re: Review Request 50069: Docker based dev environment for Log Search

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/#review142743 --- Ship it!

Re: Review Request 50069: Docker based dev environment for Log Search

> On July 15, 2016, 3:08 p.m., Nate Cole wrote: > > ambari-logsearch/docker/test-logs/zookeeper/zookeeper-test-log.log, lines > > 1-3 > > > > > > Checking in a log file? > > Oliver Szabo wrote: > That is a test

Re: Review Request 50185: [AMBARI-17786] Enable csrf protection and add superuser in livy

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50185/ --- (Updated July 19, 2016, 12:22 p.m.) Review request for Ambari and Sumit

Re: Review Request 50185: [AMBARI-17786] Enable csrf protection and add superuser

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50185/ --- (Updated July 19, 2016, 12:20 p.m.) Review request for Ambari and Sumit

Re: Review Request 50189: Getting error while executing query for adding udf jar

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50189/#review142737 --- Ship it! Ship It! - Pallav Kulshreshtha On July 19, 2016,

Re: Review Request 50189: Getting error while executing query for adding udf jar

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50189/#review142736 --- Ship it! Ship It! - Gaurav Nagar On July 19, 2016, 12:13

Re: Review Request 50189: Getting error while executing query for adding udf jar

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50189/#review142735 --- Ship it! Ship It! - Ashwin Rajeev On July 19, 2016, 12:13

Review Request 50189: Getting error while executing query for adding udf jar

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50189/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,

Re: Review Request 50087: Ambari should have a script to add new repository and service to existing stack

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50087/#review142730 --- I really like the ability to add new repos. I'm not so certain

Re: Review Request 50069: Docker based dev environment for Log Search

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- (Updated July 19, 2016, 11:23 a.m.) Review request for Ambari, Jonathan

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

> On July 15, 2016, 1:54 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java, > > line 76 > > > > > > Document why this lock is necessary/important. Done.

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

> On July 18, 2016, 3:57 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java, > > line 76 > > > > > > Important to docuemnt why this is needed. Done. > On July

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49891/ --- (Updated July 19, 2016, 1:54 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

> On July 18, 2016, 9:35 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java, > > line 142 > > > > > > Have you considered using a ConcurrentHashMap instead?

Re: Review Request 50069: Docker based dev environment for Log Search

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/#review142714 --- Ship it! Ship It! - Dharmesh Makwana On July 18, 2016, 1:28

Re: Review Request 50167: AMBARI-17779 : Fix enable/disable normalizer issue in AMS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50167/#review142713 --- Ship it! Ship It! - Dmytro Sen On Июль 18, 2016, 11:59

Review Request 50185: [AMBARI-17786] Enable csrf protection in livy by default

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50185/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-17786

Re: Review Request 50157: HST agent logs not parsed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50157/#review142707 --- Ship it! Ship It! - Miklos Gergely On July 18, 2016, 8:09

Re: Review Request 50100: AMBARI-17754 - Issue with modify configs for hive service

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50100/#review142701 --- Ship it! Ship It! - Andriy Babiichuk On Июль 16, 2016,

Review Request 50180: AMBARI-17784 AMS Storm Sink: remove redundant information from kafka offset metrics on storm-kafka (including AMBARI-17725)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50180/ --- Review request for Ambari, Aravindan Vijayan, Sriharsha Chintalapani, and Sid