Re: Review Request 55707: AMBARI-19622: Need abilities to add a custom action node and import a workflow xml with custom action

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55707/#review162251 --- Ship it! Ship It! - belliraj hb On Jan. 19, 2017, 6:54 a.m.,

Review Request 55707: AMBARI-19622: Need abilities to add a custom action node and import a workflow xml with custom action

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55707/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and Palla

Re: Review Request 55609: Add Livy session recovery configurations in Ambari

> On Jan. 19, 2017, 4:59 a.m., bikassaha wrote: > > Should the URL for recovery be something like > > /apps/{livy|spark}/livy-recovery? Having /livy-recovery as a top level HDFS > > dir seems distracting. It will not be in the top level HDFS dirs. It will be in User livy's current home direct

Re: Review Request 55663: Add support for call context to the stack advisor

> On Jan. 18, 2017, 9:31 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java, > > line 210 > > > > > > How come this does not t

Re: Review Request 55609: Add Livy session recovery configurations in Ambari

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55609/#review162240 --- Should the URL for recovery be something like /apps/{livy|spark}/l

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

> On 一月 19, 2017, 1:39 a.m., Alejandro Fernandez wrote: > > Pushed to trunk, commit 990747fe7d16874ef9788b1859bd0bcc6f9e2676 > > branch-2.5, commit fa6bbfff68c7f6a61e890b04e70577c5890ec94b we also consider this feature later, which can control the display of front-end by adding tag to metainfo.

Review Request 55700: Ranger load balancer url needs to be updated for dependent properties during HA

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55700/ --- Review request for Ambari, Jaimin Jetly, Xi Wang, and Yusaku Sako. Bugs: AMBARI

Re: Review Request 55692: AMBARI-19616. Hive Server Interactive : (1). Set 'metaspacesize' based on LLAP daemon size and (2). Fix for using "--slider-placement" argument only for HDP 2.6 and onwards.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55692/#review162224 --- Ship it! Ship It! - Sumit Mohanty On Jan. 18, 2017, 11:46 p.

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6/#review162223 --- Pushed to trunk, commit 990747fe7d16874ef9788b1859bd0bcc6f9e2676 b

Re: Review Request 55698: Restarting Some Components During a Suspended Upgrade Fails Due To Missing Upgrade Parameters

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55698/#review16 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Review Request 55698: Restarting Some Components During a Suspended Upgrade Fails Due To Missing Upgrade Parameters

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

Re: Review Request 55486: AMBARI-19417. Enable hiveContext in Livy

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55486/#review162219 --- Pushed to trunk, commit 348326af8374880be69870e16be19997053f2635

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

> On Jan. 18, 2017, 6:45 p.m., Alejandro Fernandez wrote: > > Ship It! Correction, this is only intended for 2.5 and trunk (3.0) - Alejandro --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55

Re: Review Request 55486: AMBARI-19417. Enable hiveContext in Livy

> On Jan. 13, 2017, 6:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK/configuration/livy-conf.xml, > > line 79 > > > > > > This a new stack, please everywhere

Review Request 55692: AMBARI-19616. Hive Server Interactive : (1). Set 'metaspacesize' based on LLAP daemon size and (2). Fix for using "--slider-placement" argument only for HDP 2.6 and onwards.

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

Re: Review Request 53686: Stage and Request status should be persisted in the database

On Dec. 15, 2016, 4:49 p.m., Jaimin Jetly wrote: > > One thing I don't quite see here (and it could be due to the size of the > > patch) is what happens in these two cases: > > - Something goes wrong when trying to store a task's status. How does the > > system recover and mark it completed? >

Re: Review Request 53686: Stage and Request status should be persisted in the database

> On Dec. 15, 2016, 4:49 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java, > > lines 445-447 > > > > > > This could be problematic if the

Re: Review Request 53686: Stage and Request status should be persisted in the database

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53686/ --- (Updated Jan. 18, 2017, 10:33 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 55643: Blueprint installation should accept quick link profile

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55643/#review162192 --- Ship it! Ship It! - Robert Nettleton On Jan. 17, 2017, 9:30

Re: Review Request 55663: Add support for call context to the stack advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/#review162183 --- Ship it! Ship It! - Sid Wagle On Jan. 18, 2017, 5:54 a.m., S

Re: Review Request 55663: Add support for call context to the stack advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/#review162180 --- ambari-server/src/main/java/org/apache/ambari/server/api/services

Re: Review Request 55663: Add support for call context to the stack advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/#review162177 --- Ship it! Ship It! - Swapan Shridhar On Jan. 18, 2017, 5:54 a

Re: Review Request 55663: Add support for call context to the stack advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/#review162175 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 18, 2017, 5:

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6/#review162173 --- Ship it! Ship It! - Jayush Luniya On Jan. 18, 2017, 6:01 a.m

Re: Review Request 55609: Add Livy session recovery configurations in Ambari

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55609/#review162172 --- Ship it! Ship It! - Jayush Luniya On Jan. 18, 2017, 2:25 a.m

Re: Review Request 55663: Add support for call context to the stack advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/#review162171 --- Ship it! Ship It! - Jayush Luniya On Jan. 18, 2017, 5:54 a.m

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/#review162164 --- Fix it, then Ship it! The patch looks fine to me overall. Just

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review162157 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/c

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6/#review162154 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 18, 2017, 6:

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162153 --- Ship it! Ship It! - Sebastian Toader On Jan. 18, 2017, 6:18

Re: Review Request 55674: AMBARI-19044 ADDENDUM Install & configure Ranger plugin components independently of Ranger admin components

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55674/#review162152 --- Ship it! Ship It! - Jayush Luniya On Jan. 18, 2017, 1:35 p.m

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162151 --- Ship it! Ship It! - Attila Magyar On Jan. 18, 2017, 5:18 p.m

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

> On Jan. 17, 2017, 2:15 p.m., Robert Levas wrote: > > Ship It! Thank you. - Amruta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review161866 ---

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

> On Jan. 18, 2017, 6:02 p.m., Di Li wrote: > > Ship It! Thank you Di, could you please help me push this to trunk? - Amruta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review16214

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review162146 --- Ship it! Ship It! - Di Li On Jan. 12, 2017, 12:33 a.m., Amru

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162144 --- Ship it! Ship It! - Robert Levas On Jan. 18, 2017, 12:18 p.m

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- (Updated Jan. 18, 2017, 5:18 p.m.) Review request for Ambari, Attila Magyar, Ro

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- (Updated Jan. 18, 2017, 5:17 p.m.) Review request for Ambari, Attila Magyar, Ro

Re: Review Request 55675: ambari-server setup failed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55675/#review162134 --- Ship it! Ship It! - Dmitro Lisnichenko On Jan. 18, 2017, 3:5

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

> On Jan. 17, 2017, 3:45 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/kerberos.json, > > line 26 > > > > > > This is essentially changing the config name. > >

Re: Review Request 55668: Log Search support livy2 server logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55668/#review162130 --- Ship it! Ship It! - Robert Nettleton On Jan. 18, 2017, 9:57

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- (Updated Jan. 18, 2017, 4:35 p.m.) Review request for Ambari, Attila Magyar, Ro

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

> On Jan. 18, 2017, 4:12 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/templates/hdfs_jaas.conf.j2, > > line 24 > > > > > > This should be > > > > ```

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162128 --- These changes must be added to HDFS 3.0.0 as well. ambari-server

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

> On Jan. 18, 2017, 3:57 p.m., Attila Magyar wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/templates/hdfs_jaas.conf.j2, > > line 24 > > > > > > is this path always the same? > >

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162125 --- Fix it, then Ship it! Ship It! ambari-server/src/main/resourc

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

> On Jan. 18, 2017, 10:57 a.m., Attila Magyar wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/templates/hdfs_jaas.conf.j2, > > line 24 > > > > > > is this path always the same? Tha

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- (Updated Jan. 18, 2017, 3:58 p.m.) Review request for Ambari, Attila Magyar, Ro

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162123 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/p

Re: Review Request 55622: Log Feeder should be able to handle metrics with https

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55622/#review162121 --- Ship it! Ship It! - Robert Nettleton On Jan. 17, 2017, 2:44

Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- Review request for Ambari, Attila Magyar, Robert Levas, and Sebastian Toader. B

Re: Review Request 55617: AMBARI-19585. Client config tags may not be saved by agent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55617/ --- (Updated Jan. 18, 2017, 4:40 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

> On Jan. 18, 2017, 7:07 a.m., Jonathan Hurley wrote: > > I see that you ran `mvn test -Dtest=ServicePropertiesTest > > -DskipPythonTests` ... but these changes are in Python ... Can you add a > > Python test to make sure that the JCEKS file copy is invoked properly? The second test I ran was

Re: Review Request 55672: AMBARI-19607:User is not able to set the start and end date for a coordinator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55672/ --- (Updated Jan. 18, 2017, 3:15 p.m.) Review request for Ambari, belliraj hb, Gaur

Re: Review Request 55643: Blueprint installation should accept quick link profile

> On Jan. 18, 2017, 1:43 p.m., Laszlo Puskas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/DefaultTimeSource.java, > > line 29 > > > > > > Why is this abstarction needed? Couldn't we simply us

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/#review162112 --- I see that you ran `mvn test -Dtest=ServicePropertiesTest -DskipPy

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/#review162111 --- This requires tests to cover the added code path. - Nate Cole O

Re: Review Request 55675: ambari-server setup failed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55675/#review162093 --- Ship it! Ship It! - Myroslav Papirkovskyy On Січ. 18, 2017,

Review Request 55675: ambari-server setup failed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55675/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Myroslav Pap

Re: Review Request 55643: Blueprint installation should accept quick link profile

> On Jan. 18, 2017, 2:43 p.m., Laszlo Puskas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/DefaultTimeSource.java, > > line 29 > > > > > > Why is this abstarction needed? Couldn't we simply us

Re: Review Request 55643: Blueprint installation should accept quick link profile

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55643/#review162065 --- ambari-server/src/main/java/org/apache/ambari/server/utils/Defaul

Re: Review Request 55643: Blueprint installation should accept quick link profile

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55643/#review162091 --- Ship it! Ship It! - Oliver Szabo On Jan. 17, 2017, 9:30 p.m.

Review Request 55674: AMBARI-19044 ADDENDUM Install & configure Ranger plugin components independently of Ranger admin components

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55674/ --- Review request for Ambari, Gautam Borad, Jayush Luniya, Sumit Mohanty, and Velmu

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 18, 2017, 1:14 p.m.) Review request for Ambari, Miklos Gergely an

Re: Review Request 55672: AMBARI-19607:User is not able to set the start and end date for a coordinator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55672/#review162084 --- Ship it! Ship It! - Gaurav Nagar On Jan. 18, 2017, 12:01 p.m

Re: Review Request 55670: AMBARI-19606:Search button is not functional in the dashboard

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55670/#review162077 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 18, 2017, 10:38

Re: Review Request 55672: AMBARI-19607:User is not able to set the start and end date for a coordinator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55672/#review162075 --- Ship it! Ship It! - belliraj hb On Jan. 18, 2017, 12:01 p.m.

Re: Review Request 55673: Add UI UT for alerts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55673/#review162074 --- Ship it! Ship It! - Andrii Tkach On Jan. 18, 2017, noon, Ole

Review Request 55672: AMBARI-19607:User is not able to set the start and end date for a coordinator

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

Review Request 55673: Add UI UT for alerts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55673/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-19611 https://issu

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 18, 2017, 11:40 a.m.) Review request for Ambari, Miklos Gergely a

Re: Review Request 55670: AMBARI-19606:Search button is not functional in the dashboard

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55670/#review162070 --- Ship it! Ship It! - Gaurav Nagar On Jan. 18, 2017, 10:38 a.m

Re: Review Request 55664: AMBARI-19602:Input logic tags are missing in the coordinator xml

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55664/#review162069 --- Ship it! Ship It! - Gaurav Nagar On Jan. 18, 2017, 7:15 a.m.

Re: Review Request 55670: AMBARI-19606:Search button is not functional in the dashboard

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55670/#review162063 --- Ship it! Ship It! - belliraj hb On Jan. 18, 2017, 10:38 a.m.

Review Request 55670: AMBARI-19606:Search button is not functional in the dashboard

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55670/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and Palla

Re: Review Request 55668: Log Search support livy2 server logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55668/#review162059 --- Ship it! Ship It! - Oliver Szabo On Jan. 18, 2017, 9:57 a.m.

Re: Review Request 55617: AMBARI-19585. Client config tags may not be saved by agent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55617/#review162057 --- Ship it! Ship It! - Sandor Magyari On Jan. 17, 2017, 1:28 p.

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 18, 2017, 10:20 a.m.) Review request for Ambari, Miklos Gergely a

Review Request 55668: Log Search support livy2 server logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55668/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs: AMBARI-1960

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55634/#review162045 --- Ship it! Ship It! - Dmitro Lisnichenko On Jan. 17, 2017, 10: