Review Request 45284: Ambari LDAP integration cannot handle LDAP directories with multiple entries for the same user

2016-03-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45284/ --- Review request for Ambari, Andrii Tkach, Dmitro Lisnichenko, Myroslav

Re: Review Request 45220: /tmp hdfs folder created with mode 0777

2016-03-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45220/#review125250 --- Ship it! Ship It! - Sebastian Toader On March 24, 2016,

Review Request 45291: When saving repository changes, confirm dialog shows message key instead of actual message

2016-03-24 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45291/ --- Review request for Ambari, Alexandr Antonenko and Andrii Tkach. Bugs:

Re: Review Request 45284: Ambari LDAP integration cannot handle LDAP directories with multiple entries for the same user

2016-03-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45284/#review125248 ---

Re: Review Request 45285: ambari-agent upstart script broken in RHEL6

2016-03-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45285/#review125247 --- Ship it! Ship It! - Dmitro Lisnichenko On March 24, 2016,

Review Request 45289: RBAC based user access to view instances are not honoured

2016-03-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45289/ --- Review request for Ambari, Jonathan Hurley, Oliver Szabo, and Swapan Shridhar.

Re: Review Request 45325: Add the Config, Start and Stop logic for: (1). Hive Server Interactive, and (2). Associate the LLAP lifecycle to it.

2016-03-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45325/#review125386 ---

Review Request 45295: AMBARI-15547: User names in the List view of Roles are displayed with URI encoding

2016-03-24 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45295/ --- Review request for Ambari, Di Li, Jaimin Jetly, and Richard Zang. Bugs:

Re: Review Request 45255: PXF should use hostname instead of localhost in pxf urls

2016-03-24 Thread bhuvnesh chaudhary
> On March 24, 2016, 4:49 a.m., Matt wrote: > > ambari-server/src/main/resources/common-services/PXF/3.0.0/package/alerts/api_status.py, > > line 136 > > > > > > Did anything break when it was localhost? > > >

Re: Review Request 45284: Ambari LDAP integration cannot handle LDAP directories with multiple entries for the same user

2016-03-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45284/#review125279 --- Fix it, then Ship it!

Re: Review Request 45252: PXF service checks fails on secured cluster without YARN

2016-03-24 Thread Lav Jain
> On March 24, 2016, 1:18 a.m., Matt wrote: > > ambari-server/src/main/resources/common-services/PXF/3.0.0/package/scripts/service_check.py, > > line 154 > > > > > > Does YARN have to be functional (green status),

Re: Review Request 44972: Improve error logging for install errors during blueprint deployments.

2016-03-24 Thread Amruta Borkar
> On March 24, 2016, 4:24 p.m., Robert Levas wrote: > > What branch is this for - trunk and branch-2.2? There are two different patches for trunk and branch-2.2 as the code has been refactored in trunk. I do not have commit righs on trunk, I request you to push it to trunk once it gets 'ship

Re: Review Request 45291: When saving repository changes, confirm dialog shows message key instead of actual message

2016-03-24 Thread Sangeeta Ravindran
> On March 24, 2016, 4:34 p.m., Alexandr Antonenko wrote: > > Ship It! Thanks Alexander. Can you please help push the fix. - Sangeeta --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 45289: RBAC based user access to view instances are not honoured

2016-03-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45289/#review125298 --- Ship it! Ship It! - Oliver Szabo On March 24, 2016, 2:59

Review Request 45303: Update HAWQ default_hash_table_bucket_number property logic

2016-03-24 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45303/ --- Review request for Ambari, Alejandro Fernandez, bhuvnesh chaudhary, Goutam

Re: Review Request 44724: AMBARI-12906: Alert notifications are created even if credential fields are left empty

2016-03-24 Thread Jonathan Hurley
> On March 22, 2016, 11:07 a.m., Jonathan Hurley wrote: > > Any unit tests to cover this? > > Qin Liu wrote: > Hi Jonathan, I didn't add unit tests to cover checking spaces but I > tested it from UI. Can I still ship it? Or, can I add it from another patch > if you really want it? :-) >

Re: Review Request 45253: AMBARI-15544: Creating multinode cluster using Blueprints fails.

2016-03-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45253/#review125295 --- Ship it! Ship It! - Nate Cole On March 24, 2016, 12:42

Re: Review Request 45301: History server start failed

2016-03-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45301/#review125306 --- Ship it! Ship It! - Vitalyi Brodetskyi On Березень 24,

Review Request 45297: AMBARI-15549: Delete service - Unable to delete a service even when it's host components are in a removable state.

2016-03-24 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45297/ --- Review request for Ambari, Jaimin Jetly, Sumit Mohanty, and Sid Wagle. Bugs:

Review Request 45301: History server start failed

2016-03-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45301/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-15565

Re: Review Request 45295: AMBARI-15547: User names in the List view of Roles are displayed with URI encoding

2016-03-24 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45295/#review125312 --- Ship it! Ship It! - Richard Zang On March 24, 2016, 4:32

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45254/ --- (Updated March 24, 2016, 6:22 p.m.) Review request for Ambari, Alejandro

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45254/#review125314 --- Ship it! Ship It! - Alejandro Fernandez On March 24, 2016,

Re: Review Request 45284: Ambari LDAP integration cannot handle LDAP directories with multiple entries for the same user

2016-03-24 Thread Robert Levas
> On March 24, 2016, 12:52 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariAuthentication.java, > > line 39 > > > > > > Is there any chance that this

Re: Review Request 45289: RBAC based user access to view instances are not honoured

2016-03-24 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45289/#review125296 --- Ship it! Ship It! - Swapan Shridhar On March 24, 2016, 2:59

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Jayush Luniya
> On March 24, 2016, 6:59 p.m., Jayush Luniya wrote: > > Ship It! Committed to trunk - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45254/#review125316

Review Request 45306: ambari-server check-database has errors after upgrade from 170 to ambari 2.2.2.0 [service SLIDER]

2016-03-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/ --- Review request for Ambari, Dmytro Sen and Myroslav Papirkovskyy. Bugs:

Re: Review Request 45306: ambari-server check-database has errors after upgrade from 170 to ambari 2.2.2.0 [service SLIDER]

2016-03-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/#review125321 --- Ship it! Fix the minor type and ship. - Sumit Mohanty On

Re: Review Request 45306: ambari-server check-database has errors after upgrade from 170 to ambari 2.2.2.0 [service SLIDER]

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/#review125328 --- Ship it! Ship It! - Alejandro Fernandez On March 24, 2016,

Re: Review Request 44724: AMBARI-12906: Alert notifications are created even if credential fields are left empty

2016-03-24 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44724/#review125330 --- Ship it! Ship It! - Richard Zang On March 20, 2016, 5:50

Re: Review Request 45306: ambari-server check-database has errors after upgrade from 170 to ambari 2.2.2.0 [service SLIDER]

2016-03-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/ --- (Updated Березень 24, 2016, 7:54 після полудня) Review request for Ambari,

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Juanjo Marron
> On March 24, 2016, 6:59 p.m., Jayush Luniya wrote: > > Ship It! > > Jayush Luniya wrote: > Committed to trunk Thanks for the review and for commiting! - Juanjo --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45254/#review125316 --- Ship it! Ship It! - Jayush Luniya On March 24, 2016, 6:22

Re: Review Request 45306: ambari-server check-database has errors after upgrade from 170 to ambari 2.2.2.0 [service SLIDER]

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/#review125325 ---

Re: Review Request 45254: Apply the stack featurization prototype detailed on AMBARI-13364 to TEZ service

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45254/#review125326 --- Ship it! Ship It! - Alejandro Fernandez On March 24, 2016,

Re: Review Request 45322: AMBARI-15571 : Disable YARN Container metrics from AMS

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45322/#review125360 --- Ship it! Ship It! - Alejandro Fernandez On March 24, 2016,

Re: Review Request 44725: After exporting blueprint from ranger enabled cluster ranger.service.https.attrib.keystore.pass is exported

2016-03-24 Thread Amruta Borkar
> On March 23, 2016, 12:31 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 2723 > > > > > > This is a great

Review Request 45322: AMBARI-15571 : Disable YARN Container metrics from AMS

2016-03-24 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45322/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Bugs:

Re: Review Request 45321: Ambari calculates stack downgrade as ABORTED under incorrect conditions. UI shows 'Downgrade paused' and button to resume downgrade even when progress is happening

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45321/ --- (Updated March 24, 2016, 10:28 p.m.) Review request for Ambari, Dmytro

Review Request 45321: Ambari calculates stack downgrade as ABORTED under incorrect conditions. UI shows 'Downgrade paused' and button to resume downgrade even when progress is hapenning

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45321/ --- Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan

Re: Review Request 45321: Ambari calculates stack downgrade as ABORTED under incorrect conditions. UI shows 'Downgrade paused' and button to resume downgrade even when progress is hapenning

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45321/#review125344 ---

Re: Review Request 45208: Cleanup LDAP sync process

2016-03-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45208/#review125231 ---

Re: Review Request 45253: AMBARI-15544: Creating multinode cluster using Blueprints fails.

2016-03-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45253/#review125232 --- Ship it!

Re: Review Request 45328: AMBARI-14451: Stack Featurize HDFS service

2016-03-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45328/#review125368 --- Ship it! Ship It! - Alejandro Fernandez On March 24, 2016,

Re: Review Request 45303: Update HAWQ default_hash_table_bucket_number property logic

2016-03-24 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45303/#review125370 ---