Re: Review Request 63937: AMBARI-22472. Ambari Upgrade 2.5 -> 2.6 : Update NodeManager's HSI identity 'llap_zk_hive' and 'llap_task_hive' to use '/HIVE/HIVE_SERVER/hive_server_hive' reference instead

2017-11-20 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63937/ --- (Updated Nov. 21, 2017, 7:28 a.m.) Review request for Ambari, Jayush Luniya,

Re: Review Request 63937: AMBARI-22472. Ambari Upgrade 2.5 -> 2.6 : Update NodeManager's HSI identity 'llap_zk_hive' and 'llap_task_hive' to use '/HIVE/HIVE_SERVER/hive_server_hive' reference instead

2017-11-20 Thread Swapan Shridhar
> On Nov. 18, 2017, 12:11 p.m., Robert Levas wrote: > > Shouldn't there be at least one updated kerberos.json file? Missed on adding them. - Added files HDP/2.5/services/YARN/kerberos.json and HDP/2.6/services/YARN/kerberos.json for "llap_zk_hive" and "llap_task_hive" changes. > On Nov. 18,

Re: Review Request 63962: Conditionally Rebuild MapReduce and Tez Tarballs with LZO if Enabled

2017-11-20 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63962/#review191550 --- Fix it, then Ship it!

Review Request 63965: New endpoint service_config_version for creating new config versions via service endpoint instead of cluster update

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

Re: Review Request 63955: Validate blueprint does not allow lzo enable without setup with license agreement.

2017-11-20 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/#review191538 --- Ship it! Ship It! - Robert Nettleton On Nov. 20, 2017, 5:06

Re: Review Request 63961: Ambari upgrade failed

2017-11-20 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63961/#review191536 ---

Review Request 63961: Ambari upgrade failed

2017-11-20 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63961/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-22469

Review Request 63962: Conditionally Rebuild MapReduce and Tez Tarballs with LZO if Enabled

2017-11-20 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63962/ --- Review request for Ambari, Dmytro Grinenko and Nate Cole. Bugs: AMBARI-22486

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-20 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/#review191533 --- Ship it! Ship It! - Sid Wagle On Nov. 20, 2017, 5:06 p.m.,

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-20 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/#review191532 --- Ship it!

Re: Review Request 63955: Validate blueprint does not allow lzo enable without setup with license agreement.

2017-11-20 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/#review191531 --- Ship it! Ship It! - Nate Cole On Nov. 20, 2017, 12:06 p.m.,

Re: Review Request 63955: Validate blueprint does not allow lzo enable without setup with license agreement.

2017-11-20 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/#review191529 --- Ship it!

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-20 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/#review191526 --- Fix it, then Ship it!

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

2017-11-20 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63935/ --- (Updated Nov. 20, 2017, 5:20 p.m.) Review request for Ambari, Jayush Luniya

Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-20 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/ --- Review request for Ambari, Andrew Onischuk and Sid Wagle. Bugs: AMBARI-22484

Review Request 63956: Use case and service based mpack selection

2017-11-20 Thread Jason Golieb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63956/ --- Review request for Ambari, Alexandr Antonenko, Andrii Tkach, Denys Buzhor,

Review Request 63955: Validate blueprint does not allow lzo enable without setup with license agreement.

2017-11-20 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/ --- Review request for Ambari, Andrew Onischuk, Robert Nettleton, and Sid Wagle.

Re: Review Request 63945: AMBARI-22393: UI changes for prompting user during registration failure for new OS family

2017-11-20 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63945/ --- (Updated Nov. 20, 2017, 2:25 p.m.) Review request for Ambari and Nate Cole.

Re: Review Request 63947: Log Search UI: implement access logs table

2017-11-20 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63947/#review191497 --- Ship it! Ship It! - Andrii Tkach On Nov. 20, 2017, 11:34

Review Request 63947: Log Search UI: implement access logs table

2017-11-20 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63947/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-22477

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

2017-11-20 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63916/#review191492 --- Thanks for reviews - Andrew Onischuk On Nov. 20, 2017, 10:35

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

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

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

2017-11-20 Thread Andrew Onischuk
> On Nov. 17, 2017, 4:25 p.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. > > Jonathan Hurley wrote: > Great catch! Looks like it does: > >

Re: Review Request 63945: AMBARI-22393: UI changes for prompting user during registration failure for new OS family

2017-11-20 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63945/ --- (Updated Nov. 20, 2017, 9:48 a.m.) Review request for Ambari and Nate Cole.

Review Request 63945: AMBARI-22393: UI changes for prompting user during registration failure for new OS family

2017-11-20 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63945/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-22393