Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review130815 ---

Re: Review Request 46742: Use additional flags for HAWQ Check

2016-04-27 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46742/#review130816 --- Please add unit tests. - Alexander Denissov On April 27, 2016,

Re: Review Request 46729: [preview] Force running service checks for services before upgrade

2016-04-27 Thread Nate Cole
> On April 27, 2016, 12:39 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/ServiceCheckValidityCheck.java, > > lines 112-123 > > > > > > I don't think we want a "break;" here

Re: Review Request 46619: AMBARI-16085 : Modify Ambari stacks for Ranger (for enabling plugins) to use service keytab for creating repositories and policies

2016-04-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46619/#review130814 --- Ship it! Ship It! - Robert Levas On April 27, 2016, 6:45

Re: Review Request 46741: Config group UI enhancements

2016-04-27 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46741/#review130817 --- ambari-web/app/templates/common/configs/service_config.hbs (line

Re: Review Request 46729: [preview] Force running service checks for services before upgrade

2016-04-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/#review130804 ---

Re: Review Request 46729: [preview] Force running service checks for services before upgrade

2016-04-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/#review130807 ---

Re: Review Request 46729: [preview] Force running service checks for services before upgrade

2016-04-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/#review130810 ---

Review Request 46745: AMBARI-16140: ambari-agent RPM yum update should backup both stack and common-services

2016-04-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46745/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-16140

Re: Review Request 46742: Use additional flags for HAWQ Check

2016-04-27 Thread Matt
> On April 27, 2016, 10:45 a.m., Alexander Denissov wrote: > > Please add unit tests. Working on it - Matt --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46742/#review130816

Re: Review Request 46745: AMBARI-16140: ambari-agent RPM yum update should backup both stack and common-services

2016-04-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46745/#review130833 --- ambari-agent/src/main/package/deb/control/preinst (line 45)

Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

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

Re: Review Request 46692: Improvements to DB consistency check

2016-04-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/ --- (Updated Квітень 27, 2016, 6:31 після полудня) Review request for Ambari,

Review Request 46750: Add dryrun API for host delete

2016-04-27 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46750/ --- Review request for Ambari, Nahappan Somasundaram, Sumit Mohanty, and Sid Wagle.

Re: Review Request 46745: AMBARI-16140: ambari-agent RPM yum update should backup both stack and common-services

2016-04-27 Thread Di Li
> On April 27, 2016, 7:44 p.m., Alejandro Fernandez wrote: > > ambari-agent/src/main/package/deb/control/preinst, line 45 > > > > > > According to your comments, it is already backing up > >

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-27 Thread Tim Thorpe
> On April 22, 2016, 11:19 p.m., Matt wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 118 > > > > > > There is some additional logic for HAWQ mentioned in > >

Re: Review Request 46460: AMBARI-16002 Integrate Red Hat Satellite option in Ambari Web

2016-04-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46460/#review130839 --- Ping. What's the status of this review? - Nate Cole On April

Re: Review Request 46745: AMBARI-16140: ambari-agent RPM yum update should backup both stack and common-services

2016-04-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46745/#review130836 --- Ship it! Ship It! - Alejandro Fernandez On April 27, 2016,

Review Request 46748: Add getDeleteDirectives to org.apache.ambari.server.api.resources.ResourceDefinition

2016-04-27 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46748/ --- Review request for Ambari, Jayush Luniya, Robert Levas, and Sumit Mohanty.

Re: Review Request 46751: Ambari metrics API call should allow for early failure

2016-04-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46751/#review130880 ---

Re: Review Request 46748: Add getDeleteDirectives to org.apache.ambari.server.api.resources.ResourceDefinition

2016-04-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46748/#review130851 --- Ship it! Ship It! - Jayush Luniya On April 27, 2016, 8:57

Re: Review Request 46752: Ambari Web Service installation does not comply with Final Attribute. (isFinal is skipped)

2016-04-27 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46752/ --- (Updated April 27, 2016, 10:02 p.m.) Review request for Ambari, Alexander

Re: Review Request 46752: Ambari Web Service installation does not comply with Final Attribute. (isFinal is skipped)

2016-04-27 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46752/ --- (Updated April 27, 2016, 11:16 p.m.) Review request for Ambari, Alexander

Re: Review Request 46750: Add dryrun API for host delete

2016-04-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46750/#review130850 --- Ship it! Ship It! - Jayush Luniya On April 27, 2016, 9:38

Review Request 46752: Ambari Web Service installation does not comply with Final Attribute. (isFinal is skipped)

2016-04-27 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46752/ --- Review request for Ambari. Bugs: AMBARI-16147

Re: Review Request 46752: Ambari Web Service installation does not comply with Final Attribute. (isFinal is skipped)

2016-04-27 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46752/#review130865 --- Ship it! Ship It! - Lav Jain On April 27, 2016, 11:16 p.m.,

Review Request 46751: Ambari metrics API call should allow for early failure

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

Re: Review Request 46714: Prevent Views From Causing a Loss of Service For Ambari

2016-04-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46714/#review130861 --- Ship it! Ship It! - Jayush Luniya On April 26, 2016, 10:03

Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46760/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Sumit

Re: Review Request 46703: Reduce error logs on the Sink path if AMS is down

2016-04-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46703/#review130881 --- Ship it! Ship It! - Sid Wagle On April 26, 2016, 3:13 p.m.,

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46760/#review130878 --- Ship it! Fix the issues and ship. - Sumit Mohanty On April

Re: Review Request 46751: Ambari metrics API call should allow for early failure

2016-04-27 Thread Sid Wagle
> On April 28, 2016, 4:21 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProvider.java, > > line 209 > > > > > > should the change be

Review Request 46726: Date Filter editable in History Tab

2016-04-27 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46726/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, and Nitiraj Rathore.

Re: Review Request 46475: Provide UI validation for widget_name and description fields in Create/Edit Widget pop-up.

2016-04-27 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46475/#review130733 --- Ship it! Ship It! - Andrii Tkach On April 26, 2016, 9:09

Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-27 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-15612

Review Request 46728: AMBARI-16133: Zeppelin to select only tested Interpreter

2016-04-27 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46728/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Jayush

Re: Review Request 46702: AMBARI-16122 Logsearch: Property changes not getting reflected at the backend

2016-04-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46702/#review130758 --- Ship it! Ship It! - Miklos Gergely On April 26, 2016, 4:24

Re: Review Request 46702: AMBARI-16122 Logsearch: Property changes not getting reflected at the backend

2016-04-27 Thread Oliver Szabo
> On April 27, 2016, 7:23 a.m., Miklos Gergely wrote: > > Is it intentional that the words of the property names are separated by > > dots in the logsearch.properties.xml and logfeeder.properties.xml, but in > > the other files are separated by underscores? If there is some kind of > > logic

Re: Review Request 44704: CapSched View: Revamping the old UI

2016-04-27 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review130740 ---

Re: Review Request 44704: CapSched View: Revamping the old UI

2016-04-27 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review130753 --- Ship it! Ship It! - Sreenath Somarajapuram On April 27,

Re: Review Request 46703: Reduce error logs on the Sink path if AMS is down

2016-04-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46703/#review130786 --- Ship it! Ship It! - Dmitro Lisnichenko On April 26, 2016,

Re: Review Request 46709: Update LogSearch integration layer to use proper query parameter names for LogSearch

2016-04-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46709/#review130790 --- Ship it! Ship It! - Miklos Gergely On April 27, 2016, 1:49

Re: Review Request 46709: Update LogSearch integration layer to use proper query parameter names for LogSearch

2016-04-27 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46709/ --- (Updated April 27, 2016, 1:49 p.m.) Review request for Ambari, Don Bosco

Re: Review Request 46709: Update LogSearch integration layer to use proper query parameter names for LogSearch

2016-04-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46709/#review130787 --- Ship it! Ship It! - Oliver Szabo On April 27, 2016, 1:49

Review Request 46738: Converting cluster to Kerberos the views should automatically adjust and work prior to the kerberization.

2016-04-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46738/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav

Re: Review Request 46738: Converting cluster to Kerberos the views should automatically adjust and work prior to the kerberization.

2016-04-27 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46738/#review130774 --- Ship it! Ship It! - Ashwin Rajeev On April 27, 2016, 12:49

Re: Review Request 46738: Converting cluster to Kerberos the views should automatically adjust and work prior to the kerberization.

2016-04-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46738/ --- (Updated April 27, 2016, 2:16 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 46740: Atlas Integration : set atlas.cluster.name in hive-site

2016-04-27 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46740/ --- (Updated April 27, 2016, 2:23 p.m.) Review request for Ambari, John Speidel