Review Request 63937: AMBARI-22472. Ambari Upgrade 2.5 -> 2.6 : Update NodeManager's HSI identity 'llap_zk_hive' to use '/HIVE/HIVE_SERVER/hive_server_hive' reference instead of creating the same iden

2017-11-17 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63937/ --- Review request for Ambari, Jayush Luniya, Madhuvanthi Radhakrishnan, and Robert

Re: Review Request 63935: AMBARI-22470 : Refine Metric Definition Service and AD Query service.

2017-11-17 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63935/#review191437 --- Fix it, then Ship it! ambari-metrics/ambari-metrics-anomaly-de

Review Request 63935: AMBARI-22470 : Refine Metric Definition Service and AD Query service.

2017-11-17 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63935/ --- Review request for Ambari, Jayush Luniya and Sid Wagle. Bugs: AMBARI-22470

Re: Review Request 63923: Unable to enable hdfs, hive plugin in ranger

2017-11-17 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63923/#review191389 --- Ship it! Ship It! - Jayush Luniya On Nov. 17, 2017, 6:01 p.m

Review Request 63923: Unable to enable hdfs, hive plugin in ranger

2017-11-17 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63923/ --- Review request for Ambari, Jayush Luniya, Swapan Shridhar, and Vitalyi Brodetsky

Re: Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Jonathan Hurley
> On Nov. 17, 2017, 11:25 a.m., Nate Cole wrote: > > Please also verify if BigInsight stacks are affected by this change. If > > that is the case, then a separate JIRA should be opened to address that. Great catch! Looks like it does: https://github.com/apache/ambari/blob/branch-2.5/ambari-ser

Re: Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/#review191346 --- Please also verify if BigInsight stacks are affected by this chang

Re: Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/#review191345 --- We should add python tests for enabled and disabled LZO for covera

Re: Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/#review191344 --- Fix it, then Ship it! ambari-common/src/main/python/resource_m

Re: Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/#review191339 --- Added summary of changes to review description. Please read it for

Review Request 63916: YARN, MapReduce2, Hive, and Oozie Should Conditionally Install LZO

2017-11-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/ --- Review request for Ambari, Attila Doroszlai, Myroslav Papirkovskyy, Nate Cole, a

Re: Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-17 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/#review191337 --- Ship it! Ship It! - Attila Magyar On Nov. 17, 2017, 12:48 p.

Re: Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-17 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/#review191326 --- Ship it! Ship It! - Eugene Chekanskiy On Лис. 17, 2017, 12:4

Re: Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/#review191324 --- Ship it! Ship It! - Robert Levas On Nov. 17, 2017, 7:48 a.m.

Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-17 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/ --- Review request for Ambari, Robert Levas and Vishal Suvagia. Bugs: AMBARI-22463

Re: Review Request 63880: VDF defined HDP-GPL repo should be tagged appropriately

2017-11-17 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63880/#review191321 --- Ship it! Ship It! - Dmitro Lisnichenko On Nov. 16, 2017, 5:4

Re: Review Request 63876: ambari-server upgrade to 2.6.1 should surface the GPL agreement

2017-11-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63876/ --- (Updated Nov. 17, 2017, 11:38 a.m.) Review request for Ambari, Attila Doroszlai