Re: Review Request 64325: MD interpreter fails with NPE (Zeppelin)

2017-12-04 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64325/#review192813 --- Ship it! Ship It! - Gaurav Nagar On Dec. 5, 2017, 4:42

Review Request 64325: MD interpreter fails with NPE (Zeppelin)

2017-12-04 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64325/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit

Review Request 64316: Messages for some services during PU package installation indicate circular dependency

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

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

2017-12-04 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63965/#review192775 --- Ship it! Ship It! - Swapan Shridhar On Nov. 22, 2017, 5:47

Re: Review Request 64315: Hive Client restart fails: hadooplzo package not present in iop repos

2017-12-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64315/#review192776 --- Ship it! Ship It! - Jonathan Hurley On Dec. 4, 2017, 3:33

Re: Review Request 64305: Ambari should not force accounts created in IPA to be added a user named 'ambari-managed-principals'

2017-12-04 Thread Bolke de Bruin
> On dec 4, 2017, 6:14 p.m., Bolke de Bruin wrote: > > DISCLAIMER: It is a long time that i have looked at this code. > > > > The reason for putting it in a separate group in ipa was to be able to set > > certain security parameters (no expiry of passwords). This change requires > > to have

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-04 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64311/ --- (Updated Dec. 4, 2017, 8:08 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-04 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64311/#review192750 --- Ship it! Ship It! - Tim Thorpe On Dec. 4, 2017, 7:20 p.m.,

Re: Review Request 64310: Storm service check failed during PU due to CNF StormAtlasHook

2017-12-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64310/#review192748 --- Ship it! Ship It! - Jonathan Hurley On Dec. 4, 2017, 2:14

Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-04 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64311/ --- Review request for Ambari, Jonathan Hurley and Tim Thorpe. Bugs: AMBARI-22574

Review Request 64310: Storm service check failed during PU due to CNF StormAtlasHook

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64310/ --- Review request for Ambari and Jonathan Hurley. Bugs: AMBARI-22587

Re: Review Request 64308: Remove Beacon conf-select Changes Since It's Not Part of HDP

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64308/#review192739 --- Ship it! Ship It! - Nate Cole On Dec. 4, 2017, 12:57 p.m.,

Re: Review Request 64305: Ambari should not force accounts created in IPA to be added a user named 'ambari-managed-principals'

2017-12-04 Thread Robert Levas
> On Dec. 4, 2017, 1:14 p.m., Bolke de Bruin wrote: > > DISCLAIMER: It is a long time that i have looked at this code. > > > > The reason for putting it in a separate group in ipa was to be able to set > > certain security parameters (no expiry of passwords). This change requires > > to have

Re: Review Request 64305: Ambari should not force accounts created in IPA to be added a user named 'ambari-managed-principals'

2017-12-04 Thread Bolke de Bruin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64305/#review192729 --- DISCLAIMER: It is a long time that i have looked at this code.

Re: Review Request 64309: Fix the wording on IPA integration requirements in the Enable Kerberos Wizard

2017-12-04 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64309/#review192726 --- Ship it! Ship It! - Aravindan Vijayan On Dec. 4, 2017, 6:09

Review Request 64309: Fix the wording on IPA integration requirements in the Enable Kerberos Wizard

2017-12-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64309/ --- Review request for Ambari, Aleksandr Kovalenko, Attila Magyar, Balázs Bence

Re: Review Request 64302: PERF 1.0 stack deployment failure in ambari-2.6.1.

2017-12-04 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64302/#review192725 --- Ship it! Ship It! - Sid Wagle On Dec. 4, 2017, 4:15 p.m.,

Re: Review Request 64305: Ambari should not force accounts created in IPA to be added a user named 'ambari-managed-principals'

2017-12-04 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64305/#review192724 --- Ship it! Ship It! - Aravindan Vijayan On Dec. 4, 2017, 5:02

Review Request 64308: Remove Beacon conf-select Changes Since It's Not Part of HDP

2017-12-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64308/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-22586

Re: Review Request 64225: Implemented additional functionality on the Configure Download screen.

2017-12-04 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64225/#review192710 --- Ship it! Ship It! - Andrii Tkach On Dec. 1, 2017, 4:28

Re: Review Request 64261: Update LDAP Authentication process to work with improved user management facility

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64261/#review192707 --- Ship it!

Re: Review Request 64204: zkfc show no alerts when zkfc is down

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64204/#review192706 --- ambari-web/app/mappers/alert_definition_summary_mapper.js Lines

Re: Review Request 64242: AMBARI-22476: Modify bootstrap API to pass Ambari repo URL from UI and retrieve Host OS types

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64242/#review192705 ---

Re: Review Request 63516: Remove properties.json And Switch To Adding Properties to ResourceProviders Dynamically

2017-12-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63516/#review192704 --- Ship it! Ship It! - Nate Cole On Dec. 4, 2017, 10:30 a.m.,

Review Request 64302: PERF 1.0 stack deployment failure in ambari-2.6.1.

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

Re: Review Request 64276: hive2 queries fails after adding any service to the cluster

2017-12-04 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64276/#review192700 --- Ship it! Ship It! - Yusaku Sako On Dec. 2, 2017, 8:19 p.m.,

Re: Review Request 63516: Remove properties.json And Switch To Adding Properties to ResourceProviders Dynamically

2017-12-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63516/ --- (Updated Dec. 4, 2017, 5:30 p.m.) Review request for Ambari, Jonathan Hurley