Review Request 52369: AMBARI-12263: Support PAM as authentication mechanism for accessing Ambari UI/REST

2016-09-28 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-12263

Re: Review Request 51702: Ambari UI: Memory leak while adding and removing property

2016-09-28 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51702/#review150758 --- Ship it! Ship It! - Zhe (Joe) Wang On Sept. 27, 2016, 11:42

Re: Review Request 51815: Introduce conditional dependencies in stack defition to handle blueprint validation gracefully

2016-09-28 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51815/#review150734 --- Fix it, then Ship it! Overall, the patch looks fine to me.

Re: Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52345/#review150731 --- Ship it! Ship It! - Nate Cole On Sept. 28, 2016, 8:50 a.m.,

Re: Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Nate Cole
> On Sept. 28, 2016, 9:54 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertServiceStateListener.java, > > line 100 > > > > > > Was adding `m_` a

Review Request 52355: Add HDP-2.6 stack deriving from 2.5

2016-09-28 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52355/ --- Review request for Ambari, Sumit Mohanty, Sid Wagle, and Vitalyi Brodetskyi.

Re: Review Request 52353: Revert db optimization in UT

2016-09-28 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52353/#review150721 --- Ship it! Ship It! - Dmytro Sen On Сен. 28, 2016, 3:58 п.п.,

Review Request 52353: Revert db optimization in UT

2016-09-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52353/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.

Re: Review Request 52346: Fix Authentication data is not available error in ConfigGroupResourceProviderTest

2016-09-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52346/#review150715 --- Ship it! Ship It! - Jonathan Hurley On Sept. 28, 2016, 9:42

Re: Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Jonathan Hurley
> On Sept. 28, 2016, 9:54 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertServiceStateListener.java, > > line 100 > > > > > > Was adding `m_` a

Re: Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52345/#review150714 --- Ship it!

Review Request 52346: Fix Authentication data is not available error in ConfigGroupResourceProviderTest

2016-09-28 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52346/ --- Review request for Ambari, Jonathan Hurley, Nahappan Somasundaram, and Vitalyi

Re: Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52345/ --- (Updated Sept. 28, 2016, 8:50 a.m.) Review request for Ambari, Alejandro

Review Request 52345: Remove Global Cluster Lock Shared Between Business Objects

2016-09-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52345/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Robert Levas.

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-28 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51647/ --- (Updated Sept. 28, 2016, 12:44 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 52344: Ubuntu16 Requires libsnappy1v5 As a Dependency For Metrics

2016-09-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52344/#review150711 --- Ship it! Ship It! - Jonathan Hurley On Sept. 28, 2016, 8:23

Review Request 52344: Ubuntu16 Requires libsnappy1v5 As a Dependency For Metrics

2016-09-28 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52344/ --- Review request for Ambari, Aravindan Vijayan, Jonathan Hurley, and Sid Wagle.

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-28 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52266/#review150709 --- Ship it! Ship It! - Attila Doroszlai On Sept. 28, 2016,

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-28 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52266/#review150708 ---

Review Request 52340: Refactor solr layer in logsearch

2016-09-28 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52340/ --- Review request for Ambari, Miklos Gergely, Robert Nettleton, Sumit Mohanty, and

Review Request 52340: Refactor solr layer in logsearch

2016-09-28 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52340/ --- Review request for Ambari, Miklos Gergely, Robert Nettleton, Sumit Mohanty, and

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-28 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52266/ --- (Updated Sept. 28, 2016, 9:57 a.m.) Review request for Ambari, Attila

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-28 Thread Balázs Bence Sári
> On Sept. 26, 2016, 9:33 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/UpdateActiveRepoVersionOnStartup.java, > > line 97 > > > > > > When I tried this on my cluster that was

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-28 Thread Sreenath Somarajapuram
> On Sept. 23, 2016, 10:19 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/bower.json, line 16 > > > > > > Package versiosn must be fixed. > > Akhil PB wrote: >